[PATCH] D92385: [ARM] Add a pass that re-arranges blocks when there is a backwards WLS branch

[Sjoerd Meijer via Phabricator via llvm-commits]

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

Nice one, thanks, LGTM.



================
Comment at: llvm/lib/Target/ARM/ARMBlockPlacement.cpp:217
+  // Fix fall-through to the moved BB from the one that used to be before it.
+  if (BBPrevious && BBPrevious->isSuccessor(BB))
+    FixFallthrough(BBPrevious, BB);
----------------
Nit: we can remove `BBPrevious` from this check because we have asserted this is set?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92385