diff options
| author | Maciej W. Rozycki <macro@mips.com> | 2018-05-22 22:52:14 +0100 |
|---|---|---|
| committer | Maciej W. Rozycki <macro@mips.com> | 2018-05-22 22:52:14 +0100 |
| commit | 35f1fea3fcd44546a6cf074029c284c64ad25b3c (patch) | |
| tree | ea0fe05ea90c2c7d6477ad1238f6853499f10fee /gdb/i387-tdep.c | |
| parent | 7e947ad3438b2954e0bd248fd2c95a86bd9fe33b (diff) | |
| download | gdb-35f1fea3fcd44546a6cf074029c284c64ad25b3c.zip gdb-35f1fea3fcd44546a6cf074029c284c64ad25b3c.tar.gz gdb-35f1fea3fcd44546a6cf074029c284c64ad25b3c.tar.bz2 | |
gdb/x86: Fix `-Wstrict-overflow' build error in `i387_collect_xsave'
Make `i' defined within `i387_collect_xsave' unsigned, removing a
`-Werror=strict-overflow' compilation error:
.../gdb/i387-tdep.c: In function 'void i387_collect_xsave(const regcache*, int, void*, int)':
.../gdb/i387-tdep.c:1348:1: error: assuming signed overflow does not occur when assuming that (X + c) < X is always false [-Werror=strict-overflow]
i387_collect_xsave (const struct regcache *regcache, int regnum,
^
cc1plus: all warnings being treated as errors
Makefile:1610: recipe for target 'i387-tdep.o' failed
make: *** [i387-tdep.o] Error 1
seen with GCC 5.4.0, a commit 8ee22052f690 ("gdb/x86: Handle kernels
using compact xsave format") regression. While `regnum' can be -1 on
entry to the function, to mean all registers, `i' is only used with
non-negative register numbers.
gdb/
* i387-tdep.c (i387_collect_xsave): Make `i' unsigned.
Diffstat (limited to 'gdb/i387-tdep.c')
| -rw-r--r-- | gdb/i387-tdep.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/i387-tdep.c b/gdb/i387-tdep.c index 3effc35..fdd83f1 100644 --- a/gdb/i387-tdep.c +++ b/gdb/i387-tdep.c @@ -1354,7 +1354,7 @@ i387_collect_xsave (const struct regcache *regcache, int regnum, gdb_byte *p, *regs = (gdb_byte *) xsave; gdb_byte raw[I386_MAX_REGISTER_SIZE]; ULONGEST initial_xstate_bv, clear_bv, xstate_bv = 0; - int i; + unsigned int i; enum { x87_ctrl_or_mxcsr = 0x1, |