[PATCH] D66905: [X86] Don't emit unreachable stack adjustments

[Hans Wennborg via Phabricator via llvm-commits]

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

lgtm



================
Comment at: llvm/lib/Target/X86/X86FrameLowering.cpp:2555
+// non-meta instructions between MBBI and MBB.end().
+bool blockEndsInUnreachable(const MachineBasicBlock &MBB,
+                            MachineBasicBlock::const_iterator MBBI) {
----------------
nit: "blockEndIsUnreachable" feels like a more natural name to me (I almost read the current name like that before my eyes really focused)


================
Comment at: llvm/lib/Target/X86/X86FrameLowering.cpp:2560
+             [](const MachineBasicBlock *Succ) { return Succ->isEHPad(); }) &&
+         std::all_of(MBBI, MBB.end(), [](const MachineInstr &MI) {
+           return MI.isMetaInstruction();
----------------
You used llvm::all_of above. Could llvm::all_of or std::all_of be used for both?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66905