[llvm] r196996 - Move mips' datalayout computation out of line and add comments.
Richard Sandiford
rsandifo at linux.vnet.ibm.com
Wed Dec 11 03:03:19 PST 2013
Rafael Espindola <rafael.espindola at gmail.com> writes:
> + // 32 bit registers are always available and the stack is at least 64 bit
> + // aligned. On N64 64 bit registers are also available and the stack is
> + // 128 bit aligned.
> + if (ST.isABI_N64())
> + Ret += "-n32:64-S128";
> + else
> + Ret += "-n32-S64";
n32 also has 64-bit registers and a 128-bit stack alignment, so I wonder
whether this should include ST.isABI_N32()?
That's obviously a preexisting problem (if it is a problem) since your
patch doesn't change the behaviour. It just stood out more with the
new comments.
Thanks,
Richard
More information about the llvm-commits
mailing list