[polly] r296994 - Adapt to llvm change r296992 to unbreak the bots

[Michael Kruse via llvm-commits]

2017-03-06 2:12 GMT+01:00 Sanjoy Das via llvm-commits
<llvm-commits at lists.llvm.org>:
> Author: sanjoy
> Date: Sun Mar  5 19:12:16 2017
> New Revision: 296994
>
> URL: http://llvm.org/viewvc/llvm-project?rev=296994&view=rev
> Log:
> Adapt to llvm change r296992 to unbreak the bots
>
> r296992 made ScalarEvolution's CompareValueComplexity less aggressive,
> and that broke the polly test being fixed in this change.  This change
> explicitly bumps CompareValueComplexity in said test case to make it
> pass.
>
> Can someone from the polly team please can give me an idea on if this
> case is important enough to have
> scalar-evolution-max-value-compare-depth be 3 by default?

I compared the settings

-scalar-evolution-max-value-compare-depth=2
-scalar-evolution-max-value-compare-depth=3
-scalar-evolution-max-value-compare-depth=50

without seeing any consistent regressions.

Michael