[PATCH] Strength reduce intrinsics with overflow into regular arithmetic operations if possible.

Nadav Rotem nrotem at apple.com
Fri Dec 5 08:40:32 PST 2014


Everything except for the function WillNotOverflowSignedMul looks great. I also agree with Philip that the refactoring should go in as a separate patch. The first part of WillNotOverflowSignedMul looks correct. The second part (SignBits == BitWidth + 1) is more complex and I prefer that someone else would take a look. I believe that Michael Gottesman is a good candidate for reviewing this part of the code.

http://reviews.llvm.org/D6477






More information about the llvm-commits mailing list