[llvm-commits] CVS: llvm/lib/Transforms/Scalar/GVNPRE.cpp

[Chris Lattner]

On Jun 17, 2007, at 9:31 PM, Owen Anderson wrote:

> --- llvm/lib/Transforms/Scalar/GVNPRE.cpp:1.34	Fri Jun 15 19:26:54  
> 2007
> +++ llvm/lib/Transforms/Scalar/GVNPRE.cpp	Sun Jun 17 23:30:44 2007
> @@ -329,7 +329,12 @@
>              lhsValid = true;
>              break;
>            }
> -      lhsValid &= !dependsOnInvoke(BO->getOperand(0));
> +
> +      // Check for dependency on invoke insts
> +      // NOTE: This check is expensive, so don't do it if we
> +      // don't have to
> +      if (lhsValid)
> +        lhsValid = !dependsOnInvoke(BO->getOperand(0));

It's too bad C++ doesn't have an &&= operator eh? :)

-Chris