diff options
| author | Richard Henderson <rth@redhat.com> | 2012-06-27 14:30:41 -0700 |
|---|---|---|
| committer | Richard Henderson <rth@gcc.gnu.org> | 2012-06-27 14:30:41 -0700 |
| commit | ff4d1662c887e1c43fde5d14c7835210d3a37063 (patch) | |
| tree | d2bb1bd55eeccfad14b562ab5e4ac98a913df1ee /gcc | |
| parent | ae52af05e0325497661dd23b5c250d64b7d763b0 (diff) | |
| download | gcc-ff4d1662c887e1c43fde5d14c7835210d3a37063.zip gcc-ff4d1662c887e1c43fde5d14c7835210d3a37063.tar.gz gcc-ff4d1662c887e1c43fde5d14c7835210d3a37063.tar.bz2 | |
i386: Fix logic error in r188785
PR target/53749
* config/i386/i386.c (ix86_rtx_costs): Fix typo vs UNITS_PER_WORD
in 2012-06-23 change. Adjust two other DImode tests as well.
From-SVN: r189026
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/config/i386/i386.c | 8 |
2 files changed, 9 insertions, 5 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 87cc23b..344c298 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2012-06-27 Richard Henderson <rth@redhat.com> + + PR target/53749 + * config/i386/i386.c (ix86_rtx_costs): Fix typo vs UNITS_PER_WORD + in 2012-06-23 change. Adjust two other DImode tests as well. + 2012-06-27 Nick Clifton <nickc@redhat.com> * config/rx/rx.md (comparesi3_extend): Remove = modifier from diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index edfc649..aae8a4d 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -32210,9 +32210,8 @@ ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total, } else *total = cost->fabs; - return false; } - if (GET_MODE_SIZE (mode) < UNITS_PER_WORD) + else if (GET_MODE_SIZE (mode) > UNITS_PER_WORD) { if (CONST_INT_P (XEXP (x, 1))) { @@ -32441,7 +32440,7 @@ ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total, case AND: case IOR: case XOR: - if (!TARGET_64BIT && mode == DImode) + if (GET_MODE_SIZE (mode) > UNITS_PER_WORD) { *total = (cost->add * 2 + (rtx_cost (XEXP (x, 0), outer_code, opno, speed) @@ -32479,9 +32478,8 @@ ix86_rtx_costs (rtx x, int code_i, int outer_code_i, int opno, int *total, /* At least for published AMD latencies, this really is the same as the latency for a simple fpu operation like fabs. */ *total = cost->fabs; - return false; } - if (!TARGET_64BIT && mode == DImode) + else if (GET_MODE_SIZE (mode) > UNITS_PER_WORD) *total = cost->add * 2; else *total = cost->add; |