[llvm-commits] [llvm] r76982 - in /llvm/trunk: include/llvm/Transforms/IPO/InlinerPass.h lib/Transforms/IPO/Inliner.cpp lib/Transforms/Scalar/DeadStoreElimination.cpp lib/Transforms/Scalar/JumpThreading.cpp lib/Transforms/Scalar/SimplifyHalfPowrL

[Eli Friedman]

On Fri, Jul 24, 2009 at 11:13 AM, Dan Gohman<gohman at apple.com> wrote:
> @@ -117,8 +118,9 @@
>     // If this is a store-store dependence, then the previous store is dead so
>     // long as this store is at least as big as it.
>     if (StoreInst *DepStore = dyn_cast<StoreInst>(InstDep.getInst()))
> -      if (TD.getTypeStoreSize(DepStore->getOperand(0)->getType()) <=
> -          TD.getTypeStoreSize(SI->getOperand(0)->getType())) {
> +      if (!TD ||
> +          TD->getTypeStoreSize(DepStore->getOperand(0)->getType()) <=
> +          TD->getTypeStoreSize(SI->getOperand(0)->getType())) {
>         // Delete the store and now-dead instructions that feed it.
>         DeleteDeadInstruction(DepStore);
>         NumFastStores++;

This change looks suspicious.

-Eli