From 35f1c975a38e5a0d721777cb776576c169ed919e Mon Sep 17 00:00:00 2001 From: Eric Botcazou Date: Tue, 19 Nov 2002 20:13:20 +0000 Subject: [PATCH] re PR c/8588 (ICE in extract_insn, at recog.c:2148) MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 2002-11-19 �Eric Botcazou � PR c/8588 * optabs.c (expand_binop): Convert CONST_INTs in shift operations too. From-SVN: r59274 --- gcc/ChangeLog | 6 ++++++ gcc/optabs.c | 11 +++++------ 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 9e7721474cf..91f8dc98860 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2002-11-19  Eric Botcazou   + + PR c/8588 + * optabs.c (expand_binop): Convert CONST_INTs in shift + operations too. + 2002-11-19 Roger Sayle * gcse.c (gcse_emit_move_after): Correct typo in REG_EQUAL note. diff --git a/gcc/optabs.c b/gcc/optabs.c index 3852b6fe1db..1551ff8e806 100644 --- a/gcc/optabs.c +++ b/gcc/optabs.c @@ -781,9 +781,8 @@ expand_binop (mode, binoptab, op0, op1, target, unsignedp, methods) /* In case the insn wants input operands in modes different from those of the actual operands, convert the operands. It would seem that we don't need to convert CONST_INTs, but we do, so - that they're properly zero-extended or sign-extended for their - modes; shift operations are an exception, because the second - operand need not be extended to the mode of the result. */ + that they're properly zero-extended, sign-extended or truncated + for their mode. */ if (GET_MODE (op0) != mode0 && mode0 != VOIDmode) xop0 = convert_modes (mode0, @@ -796,7 +795,7 @@ expand_binop (mode, binoptab, op0, op1, target, unsignedp, methods) xop1 = convert_modes (mode1, GET_MODE (op1) != VOIDmode ? GET_MODE (op1) - : (shift_op ? mode1 : mode), + : mode, xop1, unsignedp); /* Now, if insn's predicates don't allow our operands, put them into @@ -2234,8 +2233,8 @@ expand_twoval_binop (binoptab, op0, op1, targ0, targ1, unsignedp) /* In case the insn wants input operands in modes different from those of the actual operands, convert the operands. It would seem that we don't need to convert CONST_INTs, but we do, so - that they're properly zero-extended or sign-extended for their - modes. */ + that they're properly zero-extended, sign-extended or truncated + for their mode. */ if (GET_MODE (op0) != mode0 && mode0 != VOIDmode) xop0 = convert_modes (mode0, -- 2.30.2