[llvm] r251770 - [SCEV] Fix PR25369

Tobias Grosser via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 1 18:30:00 PST 2015


On 11/02/2015 03:06 AM, Sanjoy Das via llvm-commits wrote:
> Author: sanjoy
> Date: Sun Nov  1 20:06:01 2015
> New Revision: 251770
>
> URL: http://llvm.org/viewvc/llvm-project?rev=251770&view=rev
> Log:
> [SCEV] Fix PR25369
>
> Have `getConstantEvolutionLoopExitValue` work correctly with multiple
> entry loops.
>
> As far as I can tell, `getConstantEvolutionLoopExitValue` never did the
> right thing for multiple entry loops; and before r249712 it would
> silently return an incorrect answer.  r249712 changed SCEV to fail an
> assert on a multiple entry loop, and this change fixes the underlying
> issue.

Thank you Sanjoy, both for placing the assert carefully and for fixing 
the issue it found so quickly!

Best,
Tobias


More information about the llvm-commits mailing list