[PATCH] D18619: [AArch64] Combine callee-save and local stack SP adjustment instructions.

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 12 11:05:48 PDT 2016


rengolin added a comment.

Hi Geoff,

I'm not familiar with this part of the code, so I'll leave the full review for Tim, but I think the idea is a good one, and the prologue/epilogue changes in the test look mostly good. Some inline comments, too.

cheers,
--renato


================
Comment at: lib/Target/AArch64/AArch64FrameLowering.cpp:695
@@ -674,2 +694,3 @@
 
+  assert(!(UseFP && ForceSP) && "ForceSP flag could not be honored");
   assert((isFixed || !RegInfo->needsStackRealignment(MF) || !UseFP) &&
----------------
Can't you just make:

    if (ForceSP)
      UseFP = false;
    else
      ...

?

================
Comment at: lib/Target/AArch64/AArch64FrameLowering.cpp:814
@@ -791,11 +813,3 @@
       ++i;
   }
 }
----------------
Would it be possible to get here and (size % 16 != 0)? If so, keeping the assert would be good.


http://reviews.llvm.org/D18619





More information about the llvm-commits mailing list