[PATCH] [PATCH] Fix for http://llvm.org/bugs/show_bug.cgi?id=19905
Sanjoy Das
sanjoy at playingwithpointers.com
Sun Jun 8 01:59:12 PDT 2014
================
Comment at: test/CodeGen/X86/pr19905.ll:26
@@ +25,3 @@
+
+; These are supposed to be offsets from the CFA, which is the value of
+; stack pointer just before the call. %rbx is the 14th register we
----------------
Anton Korobeynikov wrote:
> You redefined CFA register above to be %rbp. So, not, this is not stack pointer just before the call anymore.
>
> -120 is because the rule here is"take the address from %rbp and add 16 to it". %rbx is 13th register to be pushed, so we have to offset by 13 * -8 - 16 ;)
Makes sense, I'll fix the comment in the next iteration of the patch. :)
http://reviews.llvm.org/D4052
More information about the llvm-commits
mailing list