[llvm-commits] [llvm] r73746 - in /llvm/trunk: lib/Target/ARM/ARMInstrThumb.td test/CodeGen/ARM/2009-06-18-ThumbCommuteMul.ll

[Eli Friedman]

On Thu, Jun 18, 2009 at 11:52 PM, Evan Cheng<evan.cheng at apple.com> wrote:
>
> On Jun 18, 2009, at 6:43 PM, Eli Friedman wrote:
>
>> Author: efriedma
>> Date: Thu Jun 18 20:43:08 2009
>> New Revision: 73746
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=73746&view=rev
>> Log:
>> Mark a few Thumb instructions commutable; just happened to spot this
>> while experimenting.  I'm reasonably sure this is correct, but please
>> tell me if these instructions have some strange property which makes
>> this
>> change unsafe.
>
> It's right. Looks like we are missing the isCommutable markers on the
> ARM side as well.

Are there any non-Thumb instructions where it's relevant?

-Eli