[PATCH] [PATCH] Fix for http://llvm.org/bugs/show_bug.cgi?id=19905

Sanjoy Das sanjoy at playingwithpointers.com
Mon Jun 9 17:40:01 PDT 2014


>>! In D4052#9, @vadimcn wrote:
> @sanjoy, I wonder what happens with your patch in case of stack re-alignment.
> Sorry, I can't figure out how to get git to apply your diff.  Could you please try it on the following IR:
> 
>     declare void @llvm.eh.unwind.init()
> 
>     define coldcc void @calls_unwind_init() {
>       %s = alloca i32, align 64
>       call void @llvm.eh.unwind.init()
>       ret void
>     }
> 
> If CFI info for XMM registers still correct?

Hi Vadim,

With my changes the CFI info for the above snippet isn't correct, for the reason you mentioned earlier -- the xmm registers are spilled *after* the stack realignment, and I don't see how you could even correctly access them using %rbp.

http://reviews.llvm.org/D4052






More information about the llvm-commits mailing list