This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RFA: Try to include libunwind-ia64.h in libunwind-frame.h


On Fri, 10 Feb 2012 19:58:38 +0100, Pedro Alves wrote:
> The issue is that you're using $target to detect when to include
> libunwind-ia64.h.  That's broken for the --enable-targets=all, or
> --target=x86_64-foo --enable-target=ia64-xxx cases, when --target is _not_
> ia64-*.

OK, you are right, I thought it is good enough but I agree it is not.


> That is, it looks to me that the fix should be to make sure we include
> libunwind-ia64.h in ia64 relative code, no matter the $target.

The libunwind interface in GDB (libunwind-frame.c) is not ia64 specific in any
way.  And while you can have libunwind-ia64.so, libunwind-x86_64.so and others
installed in parallel libunwind-frame.c has only one instance in GDB and can
load only one of these.

Also the architecture of libunwind is not directly mapped from configure
architectures.  For example libunwind has x86, GDB has i386.


Maybe Tristan could prepare a better patch?


Thanks,
Jan


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]