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

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 8 11:12:11 PDT 2016


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.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 213 bytes
Desc: Digital signature
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160408/ce5408a4/attachment.sig>


More information about the llvm-commits mailing list