This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: RFA: Try to include libunwind-ia64.h in libunwind-frame.h
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: Tristan Gingold <gingold at adacore dot com>
- Cc: "gdb-patches at sourceware dot org ml" <gdb-patches at sourceware dot org>
- Date: Fri, 10 Feb 2012 19:27:05 +0100
- Subject: Re: RFA: Try to include libunwind-ia64.h in libunwind-frame.h
- References: <5D1CD28F-F628-475C-B6D8-5FCBF5290C63@adacore.com>
On Thu, 02 Feb 2012 17:32:39 +0100, Tristan Gingold wrote:
> Part of the issue is libunwind.h, which is not well designed for cross operation:
+
> Note that gdb doesn't define UNW_REMOTE_ONLY. So clearly in my case a wrong header will be used.
[ BTW could you wrap mails to 78-80 chars per line? ]
> I propose to fix this issue with the following patch:
It still did not build for me as it expects libunwind.h installed which
cross-build of libunwind does not install.
OK this patch instead?
For the test with libunwind HEAD I had to patch .so.7 -> .so.8 in GDB, posted
to
http://lists.nongnu.org/archive/html/libunwind-devel
as
[Libunwind-devel] .so version bump .7 -> .8 [Re: libunwind 1.0.1]
and for cross-built libunwind I had to fix and post
[Libunwind-devel] [patch] Fix undefined symbol: _Uelf64_valid_object
Tested only the cross-build + successful dlopen, not with real ia64 target,
though.
Thanks,
Jan
2012-02-10 Jan Kratochvil <jan.kratochvil@redhat.com>
Tristan Gingold <gingold@adacore.com>
* configure.ac (--with-libunwind=auto): Set enable_libunwind for
either of libunwind.h or libunwind-ia64.h (if ia64* is the target).
* configure: Regenerate.
* libunwind-frame.h: Conditionalize the file for HAVE_LIBUNWIND and
not HAVE_LIBUNWIND_H. Include libunwind-ia64.h instead of libunwind.h
if possible.
--- a/gdb/configure.ac
+++ b/gdb/configure.ac
@@ -349,8 +349,14 @@ AS_HELP_STRING([--with-libunwind], [use libunwind frame unwinding support]),
*) AC_MSG_ERROR(bad value ${withval} for GDB with-libunwind option) ;;
esac],[
AC_CHECK_HEADERS(libunwind.h libunwind-ia64.h)
- if test x"$ac_cv_header_libunwind_h" = xyes -a x"$ac_cv_header_libunwind_ia64_h" = xyes; then
+ if test x"$ac_cv_header_libunwind_h" = xyes; then
enable_libunwind=yes;
+ elif test x"$ac_cv_header_libunwind_ia64_h" = xyes; then
+ case "$target" in
+ ia64*)
+ enable_libunwind=yes;
+ ;;
+ esac
fi
])
--- a/gdb/libunwind-frame.h
+++ b/gdb/libunwind-frame.h
@@ -19,7 +19,7 @@
You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
-#ifdef HAVE_LIBUNWIND_H
+#ifdef HAVE_LIBUNWIND
struct frame_info;
struct frame_id;
@@ -29,7 +29,13 @@ struct gdbarch;
#ifndef LIBUNWIND_FRAME_H
#define LIBUNWIND_FRAME_H 1
-#include "libunwind.h"
+#if defined HAVE_LIBUNWIND_IA64_H
+# include "libunwind-ia64.h"
+#elif defined HAVE_LIBUNWIND_H
+# include "libunwind.h"
+#else
+# error "HAVE_LIBUNWIND && !HAVE_LIBUNWIND_IA64_H && !HAVE_LIBUNWIND_H"
+#endif
struct libunwind_descr
{
@@ -73,4 +79,4 @@ int libunwind_get_reg_special (struct gdbarch *gdbarch,
#endif /* libunwind-frame.h */
-#endif /* HAVE_LIBUNWIND_H */
+#endif /* HAVE_LIBUNWIND */