[PATCH] D47949: [callsitesplit] Limit the # of predecessors walk when recording condition
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 16 09:57:09 PDT 2018
fhahn added a comment.
Sorry for coming back to this just now. Unfortunately I won't have time to turn this into a forward-style analysis before the 7.0 branch. IMO it would be fine to get this in for the 7.0 branch, to limit compile time in edge cases, and creating a ticket to improve the situation. What do you think @davide ? In general, currently we are only supporting splitting a very limited set of call sites, so we would have to limit the forward-analysis to the relevant call sites, otherwise we end up doing (much) more work on average.
In any case, we should get still get a test case for the change.
Repository:
rL LLVM
https://reviews.llvm.org/D47949
More information about the llvm-commits
mailing list