[PATCH] D21561: [IPRA] Set callee saved registers to none for local function when IPRA is enabled.

Vivek Pandya via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 6 01:20:27 PDT 2016


vivekvpandya marked 4 inline comments as done.

================
Comment at: test/CodeGen/X86/ipra-local-linkage.ll:19
@@ +18,3 @@
+define void @bar() #0 {
+  %X = add i32 1, 3
+  call void asm sideeffect "movl  %r12d, $0", "{r15}~{r12}"(i32 %X) #0
----------------
mehdi_amini wrote:
> Take %X by argument?
Do you mean as reference?

================
Comment at: test/CodeGen/X86/ipra-local-linkage.ll:22
@@ +21,3 @@
+  ; As R15 is clobbered by foo() when IPRA is enabled value of R15 should be
+  ; reloaded after foo()
+  call void @foo()
----------------
mehdi_amini wrote:
> saved before and reloaded after?
yes value should be saved if register containing original value is also getting clobbered with function call to foo() but here R15 is having immediate value so it is just getting reloaded.


http://reviews.llvm.org/D21561





More information about the llvm-commits mailing list