This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PATCH, i386]: Fix PR 63947, Wrong fcmov suffix

From: Uros Bizjak <ubizjak at gmail dot com>
To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Wed, 19 Nov 2014 21:34:10 +0100
Subject: [PATCH, i386]: Fix PR 63947, Wrong fcmov suffix
Authentication-results: sourceware.org; auth=none

Hello!

Carry flag checks from overflow tests can propagate into FP cmove
instructions. However, while "c" and "nc" suffixes are allowed as
suffixes in integer cmove insns, they are not allowed in FP cmove
insns. Patch generates equivalent "b" and "nb" suffixes for FP mode.

2014-11-19  Uros Bizjak  <ubizjak@gmail.com>

    PR target/63947
    * config/i386/i386.c (put_condition_code) <case LTU, case GEU>:
    Output "b" and "nb" suffix for FP mode.

testsuite/ChangeLog:

2014-11-19  Uros Bizjak  <ubizjak@gmail.com>

    PR target/63947
    * gcc.target/i386/pr63947.c: New test.

Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}.

Patch was committed to mainline SVN and will be backported to all
active branches.

Uros.

Attachment: p.diff.txt
Description: Text document

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]