[PATCH] D47949: [callsitesplit] Limit the # of predecessors walk when recording condition

[Florian Hahn via Phabricator via llvm-commits]

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