[llvm-commits] [llvm] r164173 - in /llvm/trunk: include/llvm/Transforms/Utils/IntegerDivision.h lib/Transforms/Utils/CMakeLists.txt lib/Transforms/Utils/IntegerDivision.cpp

[Michael Ilseman]

I can certainly believe that there are better expansions when one of the values is a constant. I'll look into this as future work.

On Sep 19, 2012, at 4:32 AM, Joerg Sonnenberger <joerg at britannica.bec.de> wrote:

> On Tue, Sep 18, 2012 at 08:27:30PM -0700, Evan Cheng wrote:
>> 
>> On Sep 18, 2012, at 6:48 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
>> 
>>> On Tue, Sep 18, 2012 at 5:42 PM, Evan Cheng <evan.cheng at apple.com> wrote:
>>>>> There are a lot of missed optimization opportunities here, e.g.
>>>>> dividing by a constant; have you considered anything in this area?
>>>> 
>>>> Instcombine etc. should have optimized a lot of cases. This is intended to expand the cases that survived optimization.
>>> 
>>> Instcombine won't touch, e.g. x/3, and there are substantially faster
>>> ways to perform that operation.
>> 
>> Are those cases generic or target specific?
> 
> Completely generic. LLVM has code for that already.
> 
> Joerg
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits