[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