[PATCH] [Instcombine] Recognize test for overflow in integer multiplication.
Erik Verbruggen
erik.verbruggen at me.com
Sun Apr 6 06:32:23 PDT 2014
Could you rename the method to ProcessUMulZExtIdiom? The ProcessUAddIdiom handles a different case, and I have a patch that does the same for UMul.
Otherwise, LGTM, but bkramer has the final say.
As a follow-up, are you interested in doing the same for a signed sub and mul? (Add is already handled.) If not, I can do that.
http://reviews.llvm.org/D2814
More information about the llvm-commits
mailing list