[PATCH] D47949: [callsitesplit] Limit the # of predecessors walk when recording condition
Davide Italiano via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 8 09:35:00 PDT 2018
davide requested changes to this revision.
davide added a comment.
This revision now requires changes to proceed.
It seems like this is trying to hide some algorithmic problem in the pass.
I think we should try to fix the pass instead. @trentxintong can you provide such cases?
The reason why I say this is that we sprinkled cutoffs over the optimizer and some of them have bitten us back. Given that CallSiteSplitting is a relatively new pass (and a clean/simple one), maybe there's something we can try before giving up.
Repository:
rL LLVM
https://reviews.llvm.org/D47949
More information about the llvm-commits
mailing list