From 8659fff005fd09cebaa1af5a76962ec77cce320c Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Mon, 31 Aug 2020 16:42:42 +0930 Subject: [PATCH] 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 0d55433db36..2265271dba7 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,8 @@ +2020-08-31 Alan Modra + + PR 26497 + * config/tc-sh.c (assemble_ppi): Use unsigned variables. + 2020-08-31 Alan Modra PR 26495 diff --git a/gas/config/tc-sh.c b/gas/config/tc-sh.c index e1e509d7d2d..ea9052286a2 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; -- 2.30.2