[PATCH] D26389: [SCEV] limit recursion depth of CompareSCEVComplexity

[Daniil Fukalov via llvm-commits]

dfukalov updated the summary for this revision.
dfukalov updated this revision to Diff 77530.
dfukalov added a comment.

Hi Sanjoy,

I've updated the change and added unittest. I made it since reduced to 400 lines testcase can be compiled by ~10 sec on a fast machine without the fix, but the suggested unittest has "softhang" behavior.

For my initial big testcase recursion depth was up to 50+ an I never completed the compilation on a quite fast CPU. So I suggest to set it 32 by default.


https://reviews.llvm.org/D26389

Files:
  lib/Analysis/ScalarEvolution.cpp
  unittests/Analysis/ScalarEvolutionTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26389.77530.patch
Type: text/x-patch
Size: 9654 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161110/108a69c6/attachment.bin>