[llvm-commits] [llvm] r122574 - /llvm/trunk/lib/Transforms/Scalar/LoopIdiomRecognize.cpp

[Frits van Bommel]

On Mon, Dec 27, 2010 at 1:03 AM, Chris Lattner <sabre at nondot.org> wrote:
> -
> -    MadeChange |= processLoopStore(SI, BECount);
> +    WeakVH InstPtr;

Shouldn't InstPtr be initialized to something? (Say, SI?)

> +    if (processLoopStore(SI, BECount)) {

You're no longer setting MadeChange.

> +      // If processing the store invalidated our iterator, start over from the
> +      // head of the loop.
> +      if (InstPtr == 0)
> +        I = BB->begin();
> +    }
>   }
>
>   return MadeChange;