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

Chris Lattner clattner at apple.com
Fri Feb 19 14:32:18 PST 2010


On Feb 19, 2010, at 2:10 PM, Charles Davis wrote:

> 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.

Thanks!

-Chris



More information about the llvm-commits mailing list