From 8659fff005fd09cebaa1af5a76962ec77cce320c Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Mon, 31 Aug 2020 16:42:42 +0930 Subject: PR26497 UBSAN: tc-sh.c:2467 left shift cannot be represented PR 26497 * config/tc-sh.c (assemble_ppi): Use unsigned variables. --- gas/ChangeLog | 5 +++++ gas/config/tc-sh.c | 12 ++++++------ 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 0d55433..2265271 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,5 +1,10 @@ 2020-08-31 Alan Modra + PR 26497 + * config/tc-sh.c (assemble_ppi): Use unsigned variables. + +2020-08-31 Alan Modra + PR 26495 * config/tc-score.c (s3_apply_fix): Use unsigned variables. * config/tc-score7.c (s7_apply_fix): Likewise. diff --git a/gas/config/tc-sh.c b/gas/config/tc-sh.c index e1e509d..ea90522 100644 --- a/gas/config/tc-sh.c +++ b/gas/config/tc-sh.c @@ -2205,12 +2205,12 @@ find_cooked_opcode (char **str_p) static unsigned int assemble_ppi (char *op_end, sh_opcode_info *opcode) { - int movx = 0; - int movy = 0; - int cond = 0; - int field_b = 0; + unsigned int movx = 0; + unsigned int movy = 0; + unsigned int cond = 0; + unsigned int field_b = 0; char *output; - int move_code; + unsigned int move_code; unsigned int size; for (;;) @@ -2464,7 +2464,7 @@ assemble_ppi (char *op_end, sh_opcode_info *opcode) if (field_b) { /* Parallel processing insn. */ - unsigned long ppi_code = (movx | movy | 0xf800) << 16 | field_b; + unsigned int ppi_code = (movx | movy | 0xf800) << 16 | field_b; output = frag_more (4); size = 4; -- cgit v1.1