On Wed, Sep 7, 2011 at 10:01 PM, Jakub Jelinek <ja...@redhat.com> wrote:

>> > 2011-09-07  Jakub Jelinek  <ja...@redhat.com>
>> >
>> >        PR target/50310
>> >        * config/i386/i386.c (ix86_prepare_sse_fp_compare_args): For
>> >        TARGET_AVX return code for LTGT and UNEQ.
>> >        (ix86_expand_fp_vcond): Handle LTGT and UNEQ.
>> >
>> >        * gcc.c-torture/execute/ieee/pr50310.c: New test.
>> >        * gcc.dg/pr50310-2.c: New test.
>>
>> Please put early exit for TARGET_SSE at the beginning of
>> ix86_prepare_sse_fp_compare_args function. There is really no need to
>
> You mean for TARGET_AVX, right?

Oh, sure.

>> swap operands - and to help reload, since AVX instructions are
>> three-operand instructions.
>>
>> OK for mainline with this change.
>
> Here is the updated patch, I'll bootstrap/regtest it now.

Thanks.

Uros.

Reply via email to