Fix PR18361: Invalidate LoopDispositions after LoopSimplify hoists things.
authorAndrew Trick <atrick@apple.com>
Sat, 4 Jan 2014 05:52:49 +0000 (05:52 +0000)
committerAndrew Trick <atrick@apple.com>
Sat, 4 Jan 2014 05:52:49 +0000 (05:52 +0000)
commitc846d65f992799fb3a3b015020ed815c0b855663
treef2e647ab333db1b8f5aeae5931a7841c352e6395
parent6a603a699224eacb14c07344a1e576fb014b8f41
Fix PR18361: Invalidate LoopDispositions after LoopSimplify hoists things.

getSCEV for an ashr instruction creates an intermediate zext
expression when it truncates its operand.

The operand is initially inside the loop, so the narrow zext
expression has a non-loop-invariant loop disposition.

LoopSimplify then runs on an outer loop, hoists the ashr operand, and
properly invalidate the SCEVs that are mapped to value.

The SCEV expression for the ashr is now an AddRec with the hoisted
value as the now loop-invariant start value.

The LoopDisposition of this wide value was properly invalidated during
LoopSimplify.

However, if we later get the ashr SCEV again, we again try to create
the intermediate zext expression. We get the same SCEV that we did
earlier, and it is still cached because it was never mapped to a
Value. When we try to create a new AddRec we abort because we're using
the old non-loop-invariant LoopDisposition.

I don't have a solution for this other than to clear LoopDisposition
when LoopSimplify hoists things.

I think the long-term strategy should be to perform LoopSimplify on
all loops before computing SCEV and before running any loop opts on
individual loops. It's possible we may want to rerun LoopSimplify on
individual loops, but it should rarely do anything, so rarely require
invalidating SCEV.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@198478 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/Analysis/ScalarEvolution.h
lib/Transforms/Utils/LoopSimplify.cpp
test/Transforms/LoopSimplify/ashr-crash.ll [new file with mode: 0644]