diff options
| author | Andreas Schwab <schwab@linux-m68k.org> | 2012-11-05 21:11:04 +0000 |
|---|---|---|
| committer | Andreas Schwab <schwab@gcc.gnu.org> | 2012-11-05 21:11:04 +0000 |
| commit | 511d31d856ee3ecf3296f46aed7c8e15611793fb (patch) | |
| tree | e83690f770534b0780f3bf05985da435f3861cd5 /gcc | |
| parent | e77c9aedcda056161ff93b55c86789266c1895c9 (diff) | |
| download | gcc-511d31d856ee3ecf3296f46aed7c8e15611793fb.zip gcc-511d31d856ee3ecf3296f46aed7c8e15611793fb.tar.gz gcc-511d31d856ee3ecf3296f46aed7c8e15611793fb.tar.bz2 | |
final.c (final_scan_insn): Handle all comparison codes in non-jump and cmove insn.
* gcc/final.c (final_scan_insn) [HAVE_cc0]: Handle all comparison
codes in non-jump and cmove insn.
* gcc/testsuite/gcc.dg/torture/fp-compare.c: New testcase.
From-SVN: r193187
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 5 | ||||
| -rw-r--r-- | gcc/final.c | 43 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 4 | ||||
| -rw-r--r-- | gcc/testsuite/gcc.dg/torture/fp-compare.c | 23 |
4 files changed, 45 insertions, 30 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0d20017..d0ba942 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2012-11-05 Andreas Schwab <schwab@linux-m68k.org> + + * final.c (final_scan_insn) [HAVE_cc0]: Handle all comparison + codes in non-jump and cmove insn. + 2012-11-05 Uros Bizjak <ubizjak@gmail.com> Vladimir Yakovlev <vladimir.b.yakovlev@intel.com> diff --git a/gcc/final.c b/gcc/final.c index f69963d..fc10dd6 100644 --- a/gcc/final.c +++ b/gcc/final.c @@ -1,7 +1,7 @@ /* Convert RTL to assembler code and output it, for GNU compiler. Copyright (C) 1987, 1988, 1989, 1992, 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, - 2010, 2011 + 2010, 2011, 2012 Free Software Foundation, Inc. This file is part of GCC. @@ -2689,36 +2689,19 @@ final_scan_insn (rtx insn, FILE *file, int optimize_p ATTRIBUTE_UNUSED, else_rtx = const0_rtx; } - switch (GET_CODE (cond_rtx)) + if (COMPARISON_P (cond_rtx) + && XEXP (cond_rtx, 0) == cc0_rtx) { - case GTU: - case GT: - case LTU: - case LT: - case GEU: - case GE: - case LEU: - case LE: - case EQ: - case NE: - { - int result; - if (XEXP (cond_rtx, 0) != cc0_rtx) - break; - result = alter_cond (cond_rtx); - if (result == 1) - validate_change (insn, &SET_SRC (set), then_rtx, 0); - else if (result == -1) - validate_change (insn, &SET_SRC (set), else_rtx, 0); - else if (result == 2) - INSN_CODE (insn) = -1; - if (SET_DEST (set) == SET_SRC (set)) - delete_insn (insn); - } - break; - - default: - break; + int result; + result = alter_cond (cond_rtx); + if (result == 1) + validate_change (insn, &SET_SRC (set), then_rtx, 0); + else if (result == -1) + validate_change (insn, &SET_SRC (set), else_rtx, 0); + else if (result == 2) + INSN_CODE (insn) = -1; + if (SET_DEST (set) == SET_SRC (set)) + delete_insn (insn); } } diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 4d6a7dc..af92ce4 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2012-11-05 Andreas Schwab <schwab@linux-m68k.org> + + * gcc.dg/torture/fp-compare.c: New testcase. + 2012-11-05 Jan Hubicka <jh@suse.cz> * gcc.dg/const-1.c: Update. diff --git a/gcc/testsuite/gcc.dg/torture/fp-compare.c b/gcc/testsuite/gcc.dg/torture/fp-compare.c new file mode 100644 index 0000000..0d51dfd --- /dev/null +++ b/gcc/testsuite/gcc.dg/torture/fp-compare.c @@ -0,0 +1,23 @@ +/* { dg-do run } */ +/* Check that find_scan_insn properly handles swapped FP comparisons. */ +static double x; +static int exit_code; + +void __attribute__ ((noinline)) +check_int (int a, int b) +{ + exit_code += (a != b); +} + +int +main (void) +{ + x = 0.0; + asm ("" : "+m" (x)); + check_int (__builtin_isgreater (x, 1.0), 0); + check_int (__builtin_isgreaterequal (x, 1.0), 0); + check_int (__builtin_isless (x, 1.0), 1); + check_int (__builtin_islessequal (x, 1.0), 1); + check_int (__builtin_islessgreater (x, 1.0), 1); + return exit_code; +} |