diff options
| author | Segher Boessenkool <segher@kernel.crashing.org> | 2017-07-19 21:31:26 +0200 |
|---|---|---|
| committer | Segher Boessenkool <segher@gcc.gnu.org> | 2017-07-19 21:31:26 +0200 |
| commit | 6cdcce60ff317b9a3feca930306194f110ed0171 (patch) | |
| tree | 326944f75e8ff235797295ee34a283d240afdcde /gcc/combine.c | |
| parent | a6722e5bf26e4b795c005f259c68714c82c2ba31 (diff) | |
| download | gcc-6cdcce60ff317b9a3feca930306194f110ed0171.zip gcc-6cdcce60ff317b9a3feca930306194f110ed0171.tar.gz gcc-6cdcce60ff317b9a3feca930306194f110ed0171.tar.bz2 | |
combine: Fix for PR81423
We here have an AND of a SUBREG of an LSHIFTRT. If that SUBREG is
paradoxical, the extraction we form is the length of the size of the
inner mode, which includes some bits that should not be in the result.
Just give up in that case.
PR rtl-optimization/81423
* combine.c (make_compound_operation_int): Don't try to optimize
the AND of a SUBREG of an LSHIFTRT if that SUBREG is paradoxical.
From-SVN: r250365
Diffstat (limited to 'gcc/combine.c')
| -rw-r--r-- | gcc/combine.c | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/gcc/combine.c b/gcc/combine.c index c1eec91..8dc62b5 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -7956,18 +7956,9 @@ make_compound_operation_int (machine_mode mode, rtx *x_ptr, XEXP (inner_x0, 1), i, 1, 0, in_code == COMPARE); - if (new_rtx) - { - /* If we narrowed the mode when dropping the subreg, then - we must zero-extend to keep the semantics of the AND. */ - if (GET_MODE_SIZE (inner_mode) >= GET_MODE_SIZE (mode)) - ; - else if (SCALAR_INT_MODE_P (inner_mode)) - new_rtx = simplify_gen_unary (ZERO_EXTEND, mode, - new_rtx, inner_mode); - else - new_rtx = NULL; - } + /* If we narrowed the mode when dropping the subreg, then we lose. */ + if (GET_MODE_SIZE (inner_mode) < GET_MODE_SIZE (mode)) + new_rtx = NULL; /* If that didn't give anything, see if the AND simplifies on its own. */ |