[PATCH] D84397: [MSP430] Replace known epilogues with branches to __mspabi_func_epilog_N

Anton Korobeynikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 9 12:52:13 PDT 2020


asl added inline comments.


================
Comment at: llvm/lib/Target/MSP430/MSP430CommonEpilogueOptimizer.cpp:151
+    // Prevent ++It when It points to just removed terminator instruction
+    std::vector<MachineBasicBlock::iterator> OriginalTerminators(
+        MBB.getFirstTerminator(), MBB.end());
----------------
You don't need this. Just do std::next / std::prev prior to changing the iterators.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84397



More information about the llvm-commits mailing list