[PATCH] D114547: [AArch64] Async unwind - Adjust unwind info in AArch64LoadStoreOptimizer

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 11 11:55:24 PST 2022


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

LGTM with one minor comment



================
Comment at: llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp:1772
+maybeMoveCFI(MachineInstr &MI, MachineBasicBlock::iterator MaybeCFI) {
+  if (MaybeCFI->getOpcode() != TargetOpcode::CFI_INSTRUCTION ||
+      !(MI.getFlag(MachineInstr::FrameSetup) ||
----------------
Maybe assert MI is an add or subtract, to make clear what cases you're expecting to handle here?


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

https://reviews.llvm.org/D114547



More information about the llvm-commits mailing list