[PATCH] [Instcombine] Recognize test for overflow in integer	multiplication.
    Erik Verbruggen 
    erik.verbruggen at me.com
       
    Tue Apr  1 07:40:48 PDT 2014
    
    
  
  I also did a patch for PR4917, but yours looks more complete. I have some different tests, and I'll see how they compare against this. Will probably do that tomorrow (CEST).
  Nitpick: I applied this patch on top of r205314, and I get the following warning:
  /Users/erik/dev/clang-llvm/llvm-git/lib/Transforms/InstCombine/InstCombineCompares.cpp:2204:9: warning: ignoring return value of function declared with
        warn_unused_result attribute [-Wunused-result]
          Mask.trunc(MulWidth);
          ^~~~~~~~~~ ~~~~~~~~
http://llvm-reviews.chandlerc.com/D2814
    
    
More information about the llvm-commits
mailing list