[PATCH] D63972: [PowerPC] Do the Early Return for the li and unconditional branch

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineBlockPlacement.cpp:2738
+        // ChainBB and TBB. This is legal under the one of following conditions:
+        // 1. ChainBB has only one unconditional branch.
+        // 2. TBB has only one predecessor.
----------------
Maybe instead of "has only one unconditional branch", say "is empty except for an unconditional branch"


================
Comment at: llvm/test/CodeGen/PowerPC/block-placement-1.mir:302
   ; CHECK:      bb.1.for.body:
-  ; CHECK:        successors: %bb.2(0x7ffff800), %bb.3(0x00000800)
-  ; CHECK:        B %bb.2
+  ; CHECK-NET:    successors: %bb.3(0x00000800)
 
----------------
"CHECK-NET"?


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

https://reviews.llvm.org/D63972