[ScopDetect] Reject loop with multiple exit blocks.
commit0944661773640aa8a7d7852e5553a65a79858916
authorMichael Kruse <llvm@meinersbur.de>
Wed, 25 Apr 2018 18:53:33 +0000 (25 18:53 +0000)
committerMichael Kruse <llvm@meinersbur.de>
Wed, 25 Apr 2018 18:53:33 +0000 (25 18:53 +0000)
tree29bb4bda3b315fec820cf8b1c7d9bb75fe3e8afb
parent7cf0bcc12e358c2f94f1719bc1151a341a6497ac
[ScopDetect] Reject loop with multiple exit blocks.

The current statement domain derivation algorithm does not (always)
consider that different exit blocks of a loop can have different
conditions to be reached.

From the code

      for (int i = n; ; i-=2) {
        if (i <= 0) goto even;
        if (i <= 1) goto odd;
        A[i] = i;
      }
    even:
      A[0] = 42;
      return;
    odd:
      A[1] = 21;
      return;

Polly currently derives the following domains:

        Stmt_even_critedge
            Domain :=
                [n] -> { Stmt_even_critedge[] };
        Stmt_odd
            Domain :=
                [n] -> { Stmt_odd[] : (1 + n) mod 2 = 0 and n > 0 };

while the domain for the odd case is correct, Stmt_even is assumed to be
executed unconditionally, which is obviously wrong. While projecting out
the loop dimension in `adjustDomainDimensions`, it does not consider
that there are other exit condition that have matched before.

I don't know a how to fix this without changing a lot of code. Therefore
This patch rejects loops with multiple exist blocks to fix the
miscompile of test-suite's uuencode.

The odd condition is transformed by LLVM to

    %cmp1 = icmp eq i64 %indvars.iv, 1

such that the project_out in adjustDomainDimensions() indeed only
matches for odd n (using this condition only, we'd have an infinite loop
otherwise).

The even condition manifests as

    %cmp = icmp slt i64 %indvars.iv, 3

Because buildDomainsWithBranchConstraints() does not consider other exit
conditions, it has to assume that the induction variable will eventually
be lower than 3 and taking this exit.

IMHO we need to reuse the algorithm that determines the number of
iterations (addLoopBoundsToHeaderDomain) to determine which exit
condition applies first. It has to happen in
buildDomainsWithBranchConstraints() because the result will need to
propagate to successor BBs. Currently addLoopBoundsToHeaderDomain() just
look for union of all backedge conditions (which means leaving not the
loop here). The patch in llvm.org/PR35465 changes it to look for exit
conditions instead. This is required because there might be other exit
conditions that do not alternatively go back to the loop header.

Differential Revision: https://reviews.llvm.org/D45649

git-svn-id: https://llvm.org/svn/llvm-project/polly/trunk@330858 91177308-0d34-0410-b5e6-96231b3b80d8
15 files changed:
include/polly/ScopDetectionDiagnostic.h
lib/Analysis/ScopDetection.cpp
lib/Analysis/ScopDetectionDiagnostic.cpp
test/ScopDetect/index_from_unpredictable_loop.ll
test/ScopDetect/index_from_unpredictable_loop2.ll
test/ScopDetectionDiagnostics/loop_has_multiple_exits.ll [new file with mode: 0644]
test/ScopInfo/NonAffine/non_affine_region_guaranteed_non-entry.ll
test/ScopInfo/complex-loop-nesting.ll
test/ScopInfo/isl_trip_count_multiple_exiting_blocks.ll
test/ScopInfo/loop-multiexit-succ-cond.ll
test/ScopInfo/multiple_exiting_blocks.ll
test/ScopInfo/multiple_exiting_blocks_two_loop.ll
test/ScopInfo/schedule-const-post-dominator-walk-2.ll
test/ScopInfo/schedule-const-post-dominator-walk.ll
test/ScopInfo/switch-5.ll