[PATCH] D150560: [AArch64] Avoid RegScavenger::forward in AArch64SpeculationHardening

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64SpeculationHardening.cpp:299-302
+    if (I == MBB.begin())
+      RS.enterBasicBlock(MBB);
+    else
+      RS.backward(std::prev(I));
----------------
foad wrote:
> It is unfortunate that `RegScavenger::backward(MachineBasicBlock::iterator)` does not provide a way to step back to the state right at the start of the MBB.
Would it help to use reverse_iterator? When I first looked into doing this I think I got hung up on making reverse_iterator work just as well


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150560