[polly] r230624 - Allow non-affine control flow by default

[Tobias Grosser]

On 26.02.2015 16:12, Johannes Doerfert wrote:
> On 02/26, Tobias Grosser wrote:
>> On 26.02.2015 12:09, Johannes Doerfert wrote:
>>> Author: jdoerfert
>>> Date: Thu Feb 26 05:09:24 2015
>>> New Revision: 230624
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=230624&view=rev
>>> Log:
>>> Allow non-affine control flow by default
>>>
>>>    With the patches r230325, r230329 and r230340 we can handle non-affine
>>>    control flow in (loop-free) subregions. As all LLVM test-suite tests pass and
>>>    we get ~20% more non-trivial SCoPs, we activate it now by default.
>>
>> Our buildbots report three compile-time failures:
>>
>> http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-fast/builds/3987
>>
>> Did you verify the correctness on LNT with a non-asserts build?
> My HEAD is at:
>    commit 63329c0fb9dc14633887b121b20bc7a17dc492b7
>    Author: Johannes Doerfert <doerfert at cs.uni-saarland.de>
>    Date:   Thu Feb 26 11:09:24 2015 +0000
>
>        Allow non-affine control flow by default
>
> I build a Debug and a Release version with cmake/ninja and run lnt on
> the 3 failing tests:
>    FAIL: MultiSource/Benchmarks/mediabench/jpeg/jpeg-6a/cjpeg.compile_time
>    FAIL: MultiSource/Benchmarks/MiBench/consumer-jpeg/consumer-jpeg.compile_time
>    FAIL: MultiSource/Benchmarks/MiBench/consumer-lame/consumer-lame.compile_time
>
> However, I pass them with both, the Release as well as the Debug build.
>
> My lnt command line:
> lnt  runtest --verbose  nt --sandbox nt  --cc `which clang` --cxx `which
> clang++` --without-llvm --test-suite ~/repos/llvm-test-suite/
> --no-machdep-info pollyperf-O3-polly --multisample=1 --cflag=-O3
> --mllvm=-polly  -j 6  --only-test=...
>
> Do you have any idea why I do not get the same failures as the bot?

Not really, do you use the latest LLVM/clang versions?

Tobias