[llvm-commits] [llvm] r96686 - in /llvm/trunk: lib/CodeGen/MachineFunction.cpp lib/Target/X86/X86RegisterInfo.cpp test/CodeGen/X86/stack-align.ll

[Charles Davis]

Chris Lattner wrote:
> Hi Charles,
> 
> Random question, do we really need 5 bits to store the alignment of the stack?  Stored in log2 form, this supports stack alignment up to 2147483648 bytes.  I think that supporting up to (say) 256 or 4096 bytes is plenty, and it would be good to not use the space in the Attributes word.
You know, you're right. Five bits is too many. So as soon as the servers
come back up, I'll reduce the space allocated by a couple of bits.

Chip