[llvm] r256344 - Fix SCEV r256338.

[Sanjoy Das via llvm-commits]

JF Bastien via llvm-commits wrote:
> Author: jfb
> Date: Wed Dec 23 12:18:53 2015
> New Revision: 256344
>
> URL: http://llvm.org/viewvc/llvm-project?rev=256344&view=rev
> Log:
> Fix SCEV r256338.

Thanks for the fix.  However, it looks like

   for (auto *L : reverse(*L))
     getLoopBackedgeTakenCounts(L, Map, SE); // recurse.


compiles fine with clang 3.7.0 on OSX.  Any idea why the clang
buildbots are different?

-- Sanjoy

>
> Modified:
>      llvm/trunk/lib/Analysis/ScalarEvolution.cpp
>
> Modified: llvm/trunk/lib/Analysis/ScalarEvolution.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Analysis/ScalarEvolution.cpp?rev=256344&r1=256343&r2=256344&view=diff
> ==============================================================================
> --- llvm/trunk/lib/Analysis/ScalarEvolution.cpp (original)
> +++ llvm/trunk/lib/Analysis/ScalarEvolution.cpp Wed Dec 23 12:18:53 2015
> @@ -9453,8 +9453,8 @@ getLoopBackedgeTakenCounts(Loop *L, Veri
>       replaceSubString(OS.str(), "<nuw>", "");
>     }
>
> -  for (auto *L : reverse(*L))
> -    getLoopBackedgeTakenCounts(L, Map, SE); // recurse.
> +  for (auto *R : reverse(*L))
> +    getLoopBackedgeTakenCounts(R, Map, SE); // recurse.
>   }
>
>   void ScalarEvolution::verify() const {
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits