MIN_ALIGN is definitely 8 on 32-bit x86, at least on Windows.

[retep998]

This fixes it so the Rust allocator on Windows 32-bit doesn't think the system allocator is aligned to 16 when it is really only aligned to 8.

[rust-highfive]

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

[pnkfelix]

Hmm, is this too broad, or is it perfect ? ... we may want to double-check against some other targets.

(The reason i say it might be "perfect" is that, if i understand correctly, this can at the very worst just cause some code to get slower ... it shouldn't inject any unsoundness... so it may be fine to just take the patch...)

[alexcrichton]

https://msdn.microsoft.com/en-us/library/ycsb6wwf.aspx seems to agree, and otherwise I agree that this probably won't harm code, just perhaps make it a tad slower in some niche situations (it's pretty rare to ask for > 8 alignment anyway).

@bors: r+ 098f4fb

[bors]

⌛ Testing commit 098f4fb with merge 5854fed...

[bors]

☀️ Test successful - auto-linux-32-nopt-t, auto-linux-32-opt, auto-linux-64-debug-opt, auto-linux-64-nopt-t, auto-linux-64-opt, auto-linux-64-x-android-t, auto-linux-cross-opt, auto-linux-musl-64-opt, auto-mac-32-opt, auto-mac-64-nopt-t, auto-mac-64-opt, auto-win-gnu-32-nopt-t, auto-win-gnu-32-opt, auto-win-gnu-64-nopt-t, auto-win-gnu-64-opt, auto-win-msvc-32-opt, auto-win-msvc-64-opt