[llvm-commits] [llvm] r133285 - /llvm/trunk/lib/Transforms/InstCombine/InstCombineCasts.cpp

Duncan Sands baldrick at free.fr
Sun Jun 19 05:58:30 PDT 2011


Hi Stuart,

> +  // Insist that the amount-to-allocate not overflow.
> +  OverflowingBinaryOperator *OBI = dyn_cast<OverflowingBinaryOperator>(Val);
> +  if (OBI&&  !OBI->hasNoUnsignedWrap()) return 0;

I noticed you dropped the test for NSW.  I thought this was a problem because
front-ends generally don't ever produce NUW, though they may produce NSW...

Ciao, Duncan.



More information about the llvm-commits mailing list