aboutsummaryrefslogtreecommitdiff
path: root/gdb/producer.c
diff options
context:
space:
mode:
authorBruno Larsen <blarsen@redhat.com>2022-04-20 14:41:11 -0300
committerBruno Larsen <blarsen@redhat.com>2022-11-03 14:08:17 +0100
commite7e7469e7a31bd5a406a03aa83a1cd648f5ef30d (patch)
tree7ae6dc9bd3ce0a081b8fa5f549b506a4de450935 /gdb/producer.c
parent78cd9188d0fb3ab8b1c33b4cb54ad85ffa444192 (diff)
downloadgdb-e7e7469e7a31bd5a406a03aa83a1cd648f5ef30d.zip
gdb-e7e7469e7a31bd5a406a03aa83a1cd648f5ef30d.tar.gz
gdb-e7e7469e7a31bd5a406a03aa83a1cd648f5ef30d.tar.bz2
gdb: Fix issue with Clang CLI macros
Clang up to version 15 (current) adds macros that were defined in the command line or by "other means", according to the Dwarf specification, after the last DW_MACRO_end_file, instead of before the first DW_MACRO_start_file, as the specification dictates. When GDB reads the macros after the last file is closed, the macros never end up "in scope" and so we can't print them. This has been submitted as a bug to Clang developers (https://github.com/llvm/llvm-project/issues/54506), and PR macros/29034 was opened for GDB to keep track of this. Seeing as there is no expected date for it to be fixed, add a workaround for all current versions of Clang. The workaround detects when the main file would be closed and if the producer is Clang, and turns that operation into a noop, so we keep a reference to the current_file as those macros are read. A test case was added to confirm the functionality, and the KFAIL for running gdb.base/macro-source-path when using clang. Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=29034 Approved-By: Simon Marchi <simon.marchi@efficios.com>
Diffstat (limited to 'gdb/producer.c')
-rw-r--r--gdb/producer.c24
1 files changed, 24 insertions, 0 deletions
diff --git a/gdb/producer.c b/gdb/producer.c
index ef1dd93..f658234 100644
--- a/gdb/producer.c
+++ b/gdb/producer.c
@@ -127,6 +127,30 @@ producer_is_llvm (const char *producer)
|| startswith (producer, " F90 Flang ")));
}
+/* See producer.h. */
+
+bool
+producer_is_clang (const char *producer, int *major, int *minor)
+{
+ if (producer != nullptr && startswith (producer, "clang version "))
+ {
+ int maj, min;
+ if (major == nullptr)
+ major = &maj;
+ if (minor == nullptr)
+ minor = &min;
+
+ /* The full producer string will look something like
+ "clang version XX.X.X ..."
+ So we can safely ignore all characters before the first digit. */
+ const char *cs = producer + strlen ("clang version ");
+
+ if (sscanf (cs, "%d.%d", major, minor) == 2)
+ return true;
+ }
+ return false;
+}
+
#if defined GDB_SELF_TEST
namespace selftests {
namespace producer {