diff options
author | Richard Sandiford <richard.sandiford@linaro.org> | 2017-06-12 16:06:38 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2017-06-12 16:06:38 +0000 |
commit | e701e0b9e8c74e5e93764556a447ea334e4e9389 (patch) | |
tree | 3ddda6c404d525cc5ed663ea3b8f6470ade31c0c /gcc | |
parent | 64ff4f60d0ac97acdfd869d3655cc962923faa20 (diff) | |
download | gcc-e701e0b9e8c74e5e93764556a447ea334e4e9389.zip gcc-e701e0b9e8c74e5e93764556a447ea334e4e9389.tar.gz gcc-e701e0b9e8c74e5e93764556a447ea334e4e9389.tar.bz2 |
Fix pessimistic DImode handling in combine.c:make_field_assignment
The make_field_assignment code:
src = force_to_mode (src, mode,
GET_MODE_PRECISION (mode) >= HOST_BITS_PER_WIDE_INT
? HOST_WIDE_INT_M1U
: (HOST_WIDE_INT_1U << len) - 1,
0);
would ignore the field length len for DImode, even though DImode can be
handled using HWIs. I think the code should be testing len instead.
2017-06-12 Richard Sandiford <richard.sandiford@linaro.org>
gcc/
* combine.c (make_field_assignment): Check len rather than the mode
precision when calling force_to_mode.
From-SVN: r249128
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/combine.c | 2 |
2 files changed, 6 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d4f4dee..cc5435d 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2017-06-12 Richard Sandiford <richard.sandiford@linaro.org> + + * combine.c (make_field_assignment): Check len rather than the mode + precision when calling force_to_mode. + 2017-06-12 Georg-Johann Lay <avr@gjlay.de> Support multilibs and devices that see flash in RAM address range. diff --git a/gcc/combine.c b/gcc/combine.c index 39ef3c6..2d49bc2 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -9634,7 +9634,7 @@ make_field_assignment (rtx x) other, pos), dest); src = force_to_mode (src, mode, - GET_MODE_PRECISION (mode) >= HOST_BITS_PER_WIDE_INT + len >= HOST_BITS_PER_WIDE_INT ? HOST_WIDE_INT_M1U : (HOST_WIDE_INT_1U << len) - 1, 0); |