[PATCH] D17015: ARM: Introduce conservative load/store optimization mode

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 15 07:19:52 PST 2016


jmolloy accepted this revision.
jmolloy added a comment.
This revision is now accepted and ready to land.

This is a bit grim, but it looks generally OK. I'd prefer the name to be something more explanatory, like "-arm-cripple-load-store-optimizer" but I can see why you might not want to call it that.

James


================
Comment at: lib/Target/ARM/ARMLoadStoreOptimizer.cpp:929
@@ +928,3 @@
+/// pointers.
+static bool isConservativelyFine(const TargetSubtargetInfo &STI,
+                                 const MachineInstr &MI) {
----------------
Should this return "true" if unaligned accesses are disabled?


Repository:
  rL LLVM

http://reviews.llvm.org/D17015





More information about the llvm-commits mailing list