[llvm-commits] [llvm] r60857 - in /llvm/trunk: lib/CodeGen/SelectionDAG/DAGCombiner.cpp test/CodeGen/X86/add-with-overflow.ll

[Chris Lattner]

On Dec 10, 2008, at 3:44 PM, Eli Friedman wrote:

> On Wed, Dec 10, 2008 at 2:36 PM, Bill Wendling <isanbard at gmail.com>  
> wrote:
>> Author: void
>> Date: Wed Dec 10 16:36:00 2008
>> New Revision: 60857
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=60857&view=rev
>> Log:
>> If ADD, SUB, or MUL have an overflow bit that's used, don't do  
>> transformation on
>> them. The DAG combiner expects that nodes that are transformed have  
>> one value
>> result.
>
> This approach somehow seems a little scary; I'm not completely sure,
> but do we really want target-independent nodes producing
> platform-specific results?

No, we don't.

-Chris