On Fri, Aug 8, 2014 at 1:50 PM, Carrot Wei <car...@google.com> wrote:
> Thank you for the comment, I've updated the patch.
>
> OK for trunk and 4.9 branch?
>
>
> 2014-08-08  Guozhi Wei  <car...@google.com>
>
>         * config/rs6000/rs6000.md (*movdi_internal64): Add a new constraint.
>
>
> On Wed, Aug 6, 2014 at 7:28 PM, Segher Boessenkool
> <seg...@kernel.crashing.org> wrote:
>> On Wed, Aug 06, 2014 at 04:48:26PM -0700, Carrot Wei wrote:
>>> -   mtvsrd %x0,%1"
>>> -  [(set_attr "type" 
>>> "store,load,*,*,*,*,fpstore,fpload,fp,mfjmpr,mtjmpr,*,mftgpr,mffgpr,mftgpr,mffgpr")
>>> -   (set_attr "length" "4,4,4,4,4,20,4,4,4,4,4,4,4,4,4,4")])
>>> +   mtvsrd %x0,%1
>>> +   xxlxor %x0,%x0"
>>> +  [(set_attr "type" 
>>> "store,load,*,*,*,*,fpstore,fpload,fp,mfjmpr,mtjmpr,*,mftgpr,mffgpr,mftgpr,mffgpr,*")
>>> +   (set_attr "length" "4,4,4,4,4,20,4,4,4,4,4,4,4,4,4,4,4")])
>>
>>
>> "type" should be "vecsimple" for xxlxor, not "integer".

xxlxor requires three operands, not two:

xxlxor %x0,%x0,%x0

Okay with that change.

Thanks, David

Reply via email to