summaryrefslogtreecommitdiff
path: root/gcc/config/cris/cris.c
diff options
context:
space:
mode:
authorHans-Peter Nilsson <hp@axis.com>2020-02-06 18:12:11 +0100
committerHans-Peter Nilsson <hp@axis.com>2020-05-09 04:22:20 +0200
commita33649e66640b35fc69f7f7f47346614a12b12d9 (patch)
tree9fae7681a3be4ef5bc66a9aecc6908b22243b848 /gcc/config/cris/cris.c
parentb73bf8a14dd77531d72426beec22c8e4b80a22a7 (diff)
downloadgcc-a33649e66640b35fc69f7f7f47346614a12b12d9.tar.gz
cris: Enable "neg" to set condition codes.
While gcc seems to prefer transforming tests on the result of reversible operations, into tests on the original, it also can work with the destination, if allocated to the same register as it commonly-enough is. The re-use is easily covered in a test-case. (N.B.: the value 0x80000000 appears to be considered invalid and unimportant.) Spotted as a "microregression" in libgcc when comparing to the cc0 version. gcc: * config/cris/cris.c (cris_select_cc_mode): Return CC_NZmode for NEG too. Correct comment. * config/cris/cris.md ("<anz>neg<mode>2<setnz>"): Rename from "neg<mode>2".
Diffstat (limited to 'gcc/config/cris/cris.c')
-rw-r--r--gcc/config/cris/cris.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/gcc/config/cris/cris.c b/gcc/config/cris/cris.c
index d0807adc87f..2bad9393c6e 100644
--- a/gcc/config/cris/cris.c
+++ b/gcc/config/cris/cris.c
@@ -1531,9 +1531,9 @@ cris_select_cc_mode (enum rtx_code op, rtx x, rtx y)
return CCmode;
/* If we have a comparison that doesn't have to look at V or C, check
- operand x; if it looks like a binary operator, return CC_NZmode,
- else CCmode, so we only use CC_NZmode for the cases where we don't
- actually have both V and C valid. */
+ operand x; if it's a valid operator, return CC_NZmode, else CCmode,
+ so we only use CC_NZmode for the cases where we don't actually have
+ both V and C valid. */
if (op == EQ || op == NE || op == GTU || op == LEU
|| op == LT || op == GE)
{
@@ -1542,7 +1542,7 @@ cris_select_cc_mode (enum rtx_code op, rtx x, rtx y)
/* Mentioning the rtx_code here is required but not sufficient: the
insn also needs to be decorated with <setnz> (and the
anonymization prefix <anz> for a named pattern). */
- return e == PLUS || e == MINUS || e == MULT || e == NOT
+ return e == PLUS || e == MINUS || e == MULT || e == NOT || e == NEG
? CC_NZmode : CCmode;
}