[PATCH] D38918: Update successor after branch relaxation in ARM

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 01:33:54 PDT 2017


rovka added a comment.

This looks correct, but is there anything removing DestBB from the newly created block's successors if necessary?



================
Comment at: lib/Target/ARM/ARMConstantIslandPass.cpp:1718
+    // When we split the MBB we transferred all its successors to the newly
+    // created block so the DestBB is no longer marked as a predecessor. Add
+    // it back here.
----------------
s/predecessor/successor/


Repository:
  rL LLVM

https://reviews.llvm.org/D38918





More information about the llvm-commits mailing list