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 _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
