[PATCH v2] gdb/amd64: clean up unused variable

Enze Li enze.li@hotmail.com
Wed Aug 3 12:30:59 GMT 2022


On Tue, 2022-08-02 at 08:44 -0700, John Baldwin wrote:
> On 8/1/22 3:23 PM, Enze Li wrote:
> > When building with clang 15, I got this,
> > 
> >    CXX    amd64-tdep.o
> > amd64-tdep.c:1410:13: error: variable 'insn' set but not used[-
> > Werror,-Wunused-but-set-variable]
> >      gdb_byte *insn = insn_details->raw_insn + modrm_offset;
> >                  ^
> > 1 error generated.
> > 
> > The function that uses this variable has been removed in this
> > commit,
> 
> LGTM, and I think this is obvious enough you can push it.
> 

Thanks, John.  Pushed.

Enze



More information about the Gdb-patches mailing list