diff options
author | Walfred Tedeschi <walfred.tedeschi@intel.com> | 2017-01-27 15:19:12 +0100 |
---|---|---|
committer | Walfred Tedeschi <walfred.tedeschi@intel.com> | 2017-01-27 15:20:12 +0100 |
commit | 7005d26ac72abbf12e02947ab87dbb3804f326a8 (patch) | |
tree | 8ff4f102d8056f1f2403ff47915c589177d94d9b /gdb/amd64-nat.c | |
parent | 239b6d10954b18ce089e9c3a667e7f2df3655476 (diff) | |
download | fsf-binutils-gdb-7005d26ac72abbf12e02947ab87dbb3804f326a8.zip fsf-binutils-gdb-7005d26ac72abbf12e02947ab87dbb3804f326a8.tar.gz fsf-binutils-gdb-7005d26ac72abbf12e02947ab87dbb3804f326a8.tar.bz2 |
amd64: remove additional comparison for validity of a register number.
Second part of the && is already guaranteed in the "regnum < num_regs"
due to comparison above.
2017-01-27 Walfred Tedeschi <walfred.tedeschi@intel.com>
* amd64-nat.c (amd64_native_gregset_reg_offset): Simplify logic
that checks validity of a register number.
Diffstat (limited to 'gdb/amd64-nat.c')
-rw-r--r-- | gdb/amd64-nat.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gdb/amd64-nat.c b/gdb/amd64-nat.c index 18c8a99..680105b 100644 --- a/gdb/amd64-nat.c +++ b/gdb/amd64-nat.c @@ -65,10 +65,10 @@ amd64_native_gregset_reg_offset (struct gdbarch *gdbarch, int regnum) if (num_regs > gdbarch_num_regs (gdbarch)) num_regs = gdbarch_num_regs (gdbarch); - if (regnum < num_regs && regnum < gdbarch_num_regs (gdbarch)) - return reg_offset[regnum]; + if (regnum >= num_regs) + return -1; - return -1; + return reg_offset[regnum]; } /* Return whether the native general-purpose register set supplies |