diff options
| author | Alan Modra <amodra@gmail.com> | 2020-09-01 07:48:02 +0930 |
|---|---|---|
| committer | Alan Modra <amodra@gmail.com> | 2020-09-01 16:03:08 +0930 |
| commit | 8dbe96f07060bc41499557d2837afbfffa1aa7fc (patch) | |
| tree | 236d23173e050e2ccc00e8456d9de5f2b63ec759 /opcodes | |
| parent | 821e059c2626fb0b2ee91898b1c74ea9636c6175 (diff) | |
| download | gdb-8dbe96f07060bc41499557d2837afbfffa1aa7fc.zip gdb-8dbe96f07060bc41499557d2837afbfffa1aa7fc.tar.gz gdb-8dbe96f07060bc41499557d2837afbfffa1aa7fc.tar.bz2 | |
mep: ubsan: mep-ibld.c:1635,1645,1652 left shift of negative value
cpu/
* mep-core.cpu (f-8s8a2, f-12s4a2, f-17s16a2): Multiply signed
value by two rather than shifting left.
(f-24s5a2n): Similarly multiply signed f-24s5a2n-hi to extract.
opcodes/
* mep-ibld.c: Regenerate.
Diffstat (limited to 'opcodes')
| -rw-r--r-- | opcodes/ChangeLog | 4 | ||||
| -rw-r--r-- | opcodes/mep-ibld.c | 10 |
2 files changed, 9 insertions, 5 deletions
diff --git a/opcodes/ChangeLog b/opcodes/ChangeLog index 20e7ec3..719cf05 100644 --- a/opcodes/ChangeLog +++ b/opcodes/ChangeLog @@ -1,3 +1,7 @@ +2020-09-01 Alan Modra <amodra@gmail.com> + + * mep-ibld.c: Regenerate. + 2020-08-31 Cooper Qu <cooper.qu@linux.alibaba.com> * csky-dis.c (csky_output_operand): Assign dis_info.value for diff --git a/opcodes/mep-ibld.c b/opcodes/mep-ibld.c index 3de7ade..8f03813 100644 --- a/opcodes/mep-ibld.c +++ b/opcodes/mep-ibld.c @@ -1624,7 +1624,7 @@ mep_cgen_extract_operand (CGEN_CPU_DESC cd, { long value; length = extract_normal (cd, ex_info, insn_value, 0|(1<<CGEN_IFLD_SIGNED)|(1<<CGEN_IFLD_PCREL_ADDR), 0, 4, 11, 32, total_length, pc, & value); - value = ((((value) << (1))) + (pc)); + value = ((((value) * (2))) + (pc)); fields->f_12s4a2 = value; } break; @@ -1632,7 +1632,7 @@ mep_cgen_extract_operand (CGEN_CPU_DESC cd, { long value; length = extract_normal (cd, ex_info, insn_value, 0|(1<<CGEN_IFLD_SIGNED)|(1<<CGEN_IFLD_PCREL_ADDR), 0, 16, 16, 32, total_length, pc, & value); - value = ((((value) << (1))) + (pc)); + value = ((((value) * (2))) + (pc)); fields->f_17s16a2 = value; } break; @@ -1642,14 +1642,14 @@ mep_cgen_extract_operand (CGEN_CPU_DESC cd, if (length <= 0) break; length = extract_normal (cd, ex_info, insn_value, 0|(1<<CGEN_IFLD_PCREL_ADDR), 0, 5, 7, 32, total_length, pc, & fields->f_24s5a2n_lo); if (length <= 0) break; - FLD (f_24s5a2n) = ((((((FLD (f_24s5a2n_hi)) << (8))) | (((FLD (f_24s5a2n_lo)) << (1))))) + (pc)); + FLD (f_24s5a2n) = ((((((FLD (f_24s5a2n_hi)) * (256))) | (((FLD (f_24s5a2n_lo)) << (1))))) + (pc)); } break; case MEP_OPERAND_PCREL8A2 : { long value; length = extract_normal (cd, ex_info, insn_value, 0|(1<<CGEN_IFLD_SIGNED)|(1<<CGEN_IFLD_PCREL_ADDR), 0, 8, 7, 32, total_length, pc, & value); - value = ((((value) << (1))) + (pc)); + value = ((((value) * (2))) + (pc)); fields->f_8s8a2 = value; } break; @@ -1765,7 +1765,7 @@ mep_cgen_extract_operand (CGEN_CPU_DESC cd, { long value; length = extract_normal (cd, ex_info, insn_value, 0, 0, 9, 6, 32, total_length, pc, & value); - value = ((value) << (1)); + value = ((value) * (2)); fields->f_7u9a2 = value; } break; |