[llvm-commits] CVS: llvm/lib/Transforms/Scalar/GVNPRE.cpp
Chris Lattner
clattner at apple.com
Mon Jun 18 09:41:23 PDT 2007
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
More information about the llvm-commits
mailing list