[PATCH] D70450: [AArch64] Teach Load/Store optimizier to rename store operands for pairing.

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 5 15:07:24 PST 2019


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

LGTM!



================
Comment at: llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp:808
+  auto MBB = MI.getParent();
+  for (MachineBasicBlock::reverse_iterator I = MI.getReverseIterator(),
+                                           E = MBB->rend();
----------------
Can this fit on one line if you use `auto` instead of `MachineBasicBlock::reverse_iterator`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70450/new/

https://reviews.llvm.org/D70450





More information about the llvm-commits mailing list