[llvm-commits] [llvm] r92506 - /llvm/trunk/lib/Target/X86/X86InstrInfo.td

[Dan Gohman]

On Jan 4, 2010, at 1:56 PM, Eli Friedman wrote:

> On Mon, Jan 4, 2010 at 12:51 PM, Dan Gohman <gohman at apple.com> wrote:
>> Author: djg
>> Date: Mon Jan  4 14:51:05 2010
>> New Revision: 92506
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=92506&view=rev
>> Log:
>> Add SDNPCommutative and SDNPAssociative to several X86 target nodes.
>> This lets isel fold loads into them in more cases.
> 
> What does it mean for an instruction with multiple results to be
> marked SDNPAssociative?  Does it do the right thing here?

Hrm, you're right. It woudn't really work with the extra flags results,
if someone somehow defined a pattern where it would matter.  I've removed
the SDNPAssociative flags from those operators now.

Dan