From b80a346e3d567b9366a1da123eeae555a44ccf05 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Thu, 9 Dec 2021 19:02:38 -0500 Subject: [PATCH] Revert "gdbsupport: remove unnecessary `#ifndef IN_PROCESS_AGENT`" This reverts commit fe72c32765e1190c8a17d309fc3a7e1882d6a430. It turns out it was wrong, libinproctrace.so does build its own gdbsupport/tdesc.cc. This broke the build: make[1]: Entering directory '/home/simark/build/binutils-gdb-one-target/gdbserver' CXXLD libinproctrace.so /usr/bin/ld: gdbsupport/tdesc-ipa.o: in function `print_xml_feature::visit_pre(target_desc const*)': /home/simark/src/binutils-gdb/gdbserver/../gdbsupport/tdesc.cc:407: undefined reference to `tdesc_architecture_name(target_desc const*)' /usr/bin/ld: /home/simark/src/binutils-gdb/gdbserver/../gdbsupport/tdesc.cc:408: undefined reference to `tdesc_architecture_name(target_desc const*)' /usr/bin/ld: /home/simark/src/binutils-gdb/gdbserver/../gdbsupport/tdesc.cc:411: undefined reference to `tdesc_osabi_name(target_desc const*)' /usr/bin/ld: /home/simark/src/binutils-gdb/gdbserver/../gdbsupport/tdesc.cc:416: undefined reference to `tdesc_compatible_info_list(target_desc const*)' /usr/bin/ld: /home/simark/src/binutils-gdb/gdbserver/../gdbsupport/tdesc.cc:418: undefined reference to `tdesc_compatible_info_arch_name(std::unique_ptr > const&)' --- gdbsupport/tdesc.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gdbsupport/tdesc.cc b/gdbsupport/tdesc.cc index b2deea04613..c04f4943fc0 100644 --- a/gdbsupport/tdesc.cc +++ b/gdbsupport/tdesc.cc @@ -400,6 +400,7 @@ void print_xml_feature::visit (const tdesc_reg *r) void print_xml_feature::visit_pre (const target_desc *e) { +#ifndef IN_PROCESS_AGENT add_line (""); add_line (""); add_line (""); @@ -417,6 +418,7 @@ void print_xml_feature::visit_pre (const target_desc *e) for (const auto &c : compatible_list) add_line ("%s", tdesc_compatible_info_arch_name (c)); +#endif } void print_xml_feature::visit_post (const target_desc *e) -- 2.30.2