[llvm-commits] [llvm] r55318 - in /llvm/trunk/lib/Target/X86: X86InstrFormats.td X86InstrMMX.td

[Bill Wendling]

On Aug 26, 2008, at 2:29 PM, Evan Cheng wrote:

> On Aug 26, 2008, at 1:03 PM, Bill Wendling wrote:
>
>> Pretty sure. When I applied it, it ICEd saying "illegal instruction".
>> Removing it made it go away.
>>
>> My guess is that it used this instruction because it was marked as
>> "HasSSE2", but it shouldn't have because it does MMX stuff.
>
> Hmm... That's strange. The pattern is matching something that produces
> v1i16. The predicate doesn't change what gets matched. Is it possible
> to reproduce it and figure out what is miscompiled? I suspect the bug
> is elsewhere but it's being exposed by the change.
>
I did several tests and it looks like it's no longer failing. I'm at a  
loss as to why it failed before. I'm recommitting the patch.

-bw