[polly] r265775 - [FIX] Look through div & srem instructions in SCEVs

[Tobias Grosser via llvm-commits]

On 04/08/2016 08:12 PM, Johannes Doerfert wrote:
> On 04/08, Tobias Grosser wrote:
>> On 04/08/2016 12:25 PM, Johannes Doerfert via llvm-commits wrote:
>>> Author: jdoerfert
>>> Date: Fri Apr  8 05:25:58 2016
>>> New Revision: 265775
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=265775&view=rev
>>> Log:
>>> [FIX] Look through div & srem instructions in SCEVs
>>>
>>>    The findValues() function did not look through div & srem instructions
>>>    that were part of the argument SCEV. However, in different other
>>>    places we already look through it. This mismatch caused us to preload
>>>    values in the wrong order.
>>
>> It seems one of these commits (265775, 265776, 265777) broke our parallel
>> buildbot. I did not have time to reproduce this, but here is the link to the
>> first failing build:
>> 	
>> http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-parallel-fast/builds/17537
>>
>> Best,
>> Tobias
>
> With r265823 such errors will be caught during compile itme and with
> r265824 our build should be green again.

Thank you!

Best,
Tobias