[llvm-commits] [PATCH] Adding x32 ABI triple and separate StackSlotSize in MCAsmInfo

Michael Liao michael.liao at intel.com
Tue Jan 22 09:55:37 PST 2013


It's a good idea to add a dedicated slotsize in MCAsmInfo as X32 has
pointer size different from the stack slot size (due to PUSH/POP has
fixed 64-bit/8B operand size.) The patch looks good to me.

Yours
- Michael

On Fri, 2013-01-18 at 09:00 -0800, Eli Bendersky wrote:
> Hello,
> 
> Following David's email on llvmdev
> (http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-January/058209.html),
> here is an initial patch for adding the x32 ABI triple to LLVM.
> 
> The patch is deliberately very simple. All it does is add the new
> Triple::EnvironmentType value for the x32 ABI, and uses it on one
> place - separating stack slot size and pointer size in MCAsmInfo. All
> targets provide the same value there, but x86-64 has to behave
> differently based on the ABI.
> One place where this is important is the data alignment factor in
> DWARF frame section. I discussed this with Rafael previosuly and we
> agreed this solution makes sense.
> 
> A limitation in our llvm-dwarfdump tool currently precludes me from
> adding more tests that direct the slot size. I will try to see if I
> can patch up llvm-dwarfdump to allow this separately. Once that
> capability is in place, I will add more tests.
> 
> Note that changes based on the x32 ABI will have to eventually
> propagate into other places, but I will break it to small reviewable
> and testable chunks over time. For the full picture of where this is
> going, you can refer to the patches Michael Liao sent last year
> (http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-June/050360.html and
> other threads).
> 
> Eli
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list