Re: [PATCH] Include s12z-opc.h using ../opcodes.

["Christian Biesinger via gdb-patches" <gdb-patches@sourceware.org>]

On Thu, Aug 1, 2019 at 12:30 PM Simon Marchi <simark@simark.ca> wrote:
>
> On 2019-08-01 1:26 p.m., Christian Biesinger via gdb-patches wrote:
> > This file exists in binutils-gdb/opcodes, and nothing in Makefile.in adds this
> > to the include path.  Default builds work because in-tree readline adds
> > binutils-gdb/ to the include path (!). However, this is broken when compiling
> > with system readline.
>
> Yeah, my guess is that the include path is added when using the in-tree readline to make
> #include "readline/readline.h" work.  And it made this include work by chance.
>
> > Other files already use this way to include files in opcodes:
> > ./gdb/lm32-tdep.c:#include "../opcodes/lm32-desc.h"
> > ./gdb/or1k-tdep.h:#include "../opcodes/or1k-desc.h"
> > ./gdb/mep-tdep.c:#include "../opcodes/mep-desc.h"
> >
> > etc.
> >
> > gdb/ChangeLog:
> >
> > 2019-08-01  Christian Biesinger  <cbiesinger@google.com>
> >
> >       * s12z-tdep.c: Fix include path for s12z-opc.h.
> > ---
> >  gdb/s12z-tdep.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/gdb/s12z-tdep.c b/gdb/s12z-tdep.c
> > index a9d20e809c..a1c085cfbb 100644
> > --- a/gdb/s12z-tdep.c
> > +++ b/gdb/s12z-tdep.c
> > @@ -30,7 +30,7 @@
> >  #include "opcode/s12z.h"
> >  #include "trad-frame.h"
> >  #include "remote.h"
> > -#include "opcodes/s12z-opc.h"
> > +#include "../opcodes/s12z-opc.h"
> >
> >  /* Two of the registers included in S12Z_N_REGISTERS are
> >     the CCH and CCL "registers" which are just views into
> >
> Thanks, the patch LGTM.

Thanks, pushed.