diff options
| author | Simon Marchi <simon.marchi@ericsson.com> | 2018-08-21 10:47:47 -0400 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@ericsson.com> | 2018-08-21 10:47:47 -0400 |
| commit | a4497d2f8486502ab5140ec5168d5f13ec375155 (patch) | |
| tree | 392e73ab89566e9874d6f7ecd3d2bf216e8552c9 /gdb/m32c-tdep.c | |
| parent | 06d743b72334ad4f9297c206290a47c261aa8e5f (diff) | |
| download | binutils-gdb-a4497d2f8486502ab5140ec5168d5f13ec375155.tar.gz | |
Remove unnecessary ternary operator in m32c-tdep.c
Bug 17816 pointed out a useless use of the ternary operator:
case 0x0: sd.reg = (size == 1 ? &st->r0 : &st->r0); break;
I believe that this is right. If size is 1, the instruction refers to
part of r0, while if size is 2, the instruction refers to the whole of
r0.
gdb/ChangeLog:
PR gdb/17816
* m32c-tdep.c (m32c_decode_srcdest4): Remove unnecessary ternary
operator.
Diffstat (limited to 'gdb/m32c-tdep.c')
| -rw-r--r-- | gdb/m32c-tdep.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/m32c-tdep.c b/gdb/m32c-tdep.c index fc6277ef5c1..6fa24452da9 100644 --- a/gdb/m32c-tdep.c +++ b/gdb/m32c-tdep.c @@ -1236,7 +1236,7 @@ m32c_decode_srcdest4 (struct m32c_pv_state *st, switch (code) { - case 0x0: sd.reg = (size == 1 ? &st->r0 : &st->r0); break; + case 0x0: sd.reg = &st->r0; break; case 0x1: sd.reg = (size == 1 ? &st->r0 : &st->r1); break; case 0x2: sd.reg = (size == 1 ? &st->r1 : &st->r2); break; case 0x3: sd.reg = (size == 1 ? &st->r1 : &st->r3); break; |