[llvm-commits] [llvm] r70820 - in /llvm/trunk: lib/Transforms/Scalar/JumpThreading.cpp test/Transforms/JumpThreading/no-irreducible-loops.ll

[Nicolas Geoffray]

Hi Chris,

Chris Lattner wrote:
> +#ifdef NDEBUG
> +    SmallPtrSet<BasicBlock*, 16> LoopHeaders;
> +#else
> +    SmallSet<AssertingVH<BasicBlock>, 16> LoopHeaders;
> +#endif
>
>   

> @@ -109,15 +121,42 @@
>          DOUT << "  JT: Deleting dead block '" << BB->getNameStart()
>               << "' with terminator: " << *BB->getTerminator();
>          DeleteDeadBlock(BB);
> +        LoopHeaders.erase(BB);
>   

SmallSet is an AssertingVH, so erasing the block in the LoopHeaders 
should come before deleting the block.

Nicolas