[PATCH] D36512: [ValueTracking] Honour recursion limit
Chad Rosier via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 9 07:57:06 PDT 2017
mcrosier accepted this revision.
mcrosier added a comment.
This revision is now accepted and ready to land.
One small comment, but otherwise LGTM. Thanks, Davide.
Comment at: lib/Analysis/ValueTracking.cpp:4534
// The remaining tests are all recursive, so bail out if we hit the limit.
if (Depth == MaxDepth)
Is this check still necessary? I don't think it is, but of course, I'm the one who broke things in the first place.. :P
More information about the llvm-commits