[PATCH] D57067: [MBP] Don't move bottom block before header if it can't reduce taken branches

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 22 22:10:21 PST 2019


davidxl added inline comments.


================
Comment at: lib/CodeGen/MachineBlockPlacement.cpp:1775
+// layout the other successor below it, so it can't reduce taken branch.
+// In this case we keep its original layout.
+bool
----------------
Is there any benefit of keeping the original layout?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57067





More information about the llvm-commits mailing list