[llvm-commits] [llvm] r60399 - in /llvm/trunk/lib: Target/README.txt Transforms/Scalar/InstructionCombining.cpp

Bill Wendling isanbard at gmail.com
Tue Dec 2 04:06:20 PST 2008


On Dec 2, 2008, at 12:11 AM, Duncan Sands wrote:

>> - Remove the buggy -X/C -> X/-C transform. This isn't valid when X  
>> isn't a
>>  constant. If X is a constant, then this is folded elsewhere.
>
> why isn't this valid if X isn't a constant?  It is only invalid if
> X == minint as far as I can see (of course, it may be hard to tell
> if X == minint if X is not a constant).

Yes. That's it exactly. :-)

-bw




More information about the llvm-commits mailing list