[polly] r248667 - [FIX] Use the surrounding loop for non-affine SCoP regions

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 26 07:41:39 PDT 2015


On 09/26/2015 03:41 PM, Johannes Doerfert via llvm-commits wrote:
> Author: jdoerfert
> Date: Sat Sep 26 08:41:43 2015
> New Revision: 248667
>
> URL: http://llvm.org/viewvc/llvm-project?rev=248667&view=rev
> Log:
> [FIX] Use the surrounding loop for non-affine SCoP regions
>
>    When the whole SCoP is a non-affine region we need to use the
>    surrounding loop in the construction of the schedule as that is
>    the one that will be looked up after the schedule generation.
>
>    This fixes bug 24947

Thanks, this seems to have indeed fixed most compile-time failures we have
seen before for these minimal scops, but interestingly now most (all?) of them
turned into miscompiles:

http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-unprofitable/builds/17

Best,
Tobias


More information about the llvm-commits mailing list