[PATCH] Ensure vcond* expansion doesn't fail on x86 (PR target/50310)

Uros Bizjak ubizjak@gmail.com
Wed Sep 7 20:49:00 GMT 2011


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

>> > 2011-09-07  Jakub Jelinek  <jakub@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.



More information about the Gcc-patches mailing list