[PATCH] D18901: ARM: Use a callee save register for the swiftself parameter.

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 8 13:04:10 PDT 2016


rengolin added inline comments.

================
Comment at: lib/Target/ARM/ARMFrameLowering.cpp:914
@@ -925,1 +913,3 @@
+      bool isLiveIn = MF.getRegInfo().isLiveIn(Reg);
+      if (!isLiveIn)
         MBB.addLiveIn(Reg);
----------------
yeah, I meant it as a duplication. :)

================
Comment at: test/CodeGen/ARM/swiftself.ll:2
@@ -3,1 +1,3 @@
+; RUN: llc -verify-machineinstrs -mtriple=armv7k-apple-ios8.0 -mcpu=cortex-a7 -o - %s | FileCheck --check-prefix=CHECK --check-prefix=OPT %s
+; RUN: llc -O0 -verify-machineinstrs -mtriple=armv7k-apple-ios8.0 -mcpu=cortex-a7 -o - %s | FileCheck %s
 
----------------
The CHECK prefix is still there... already assumed by default.


Repository:
  rL LLVM

http://reviews.llvm.org/D18901





More information about the llvm-commits mailing list