[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