commit 15659f0a4ea0787d2c2194a4e3712a56ac3eff84
author Tom Tromey <tom@tromey.com> Sun Jul 04 13:33:12 2021 -0600
committer Tom Tromey <tom@tromey.com> Sat Jul 17 10:41:42 2021 -0600
tree 223f810288bca5a796599993f0f0f3b0b073f90a
parent e0ec6b1a8796127c5141647bfe88e260716d6cd4

Check for debug-types in map_symbol_filenames

map_symbol_filenames can skip type units -- in fact I think it has to,
due to the assertion at the top of dw2_get_file_names.  This may be a
regression due to the TU/CU unification patch, I did not check.

gdb/dwarf2/read.c

diff --git a/gdb/dwarf2/read.c b/gdb/dwarf2/read.c
index c730c52..9054259 100644
--- a/gdb/dwarf2/read.c
+++ b/gdb/dwarf2/read.c
@@ -4420,7 +4420,8 @@
 
   for (const auto &per_cu : per_objfile->per_bfd->all_comp_units)
     {
-      if (per_objfile->symtab_set_p (per_cu.get ()))
+      if (!per_cu->is_debug_types
+	  && per_objfile->symtab_set_p (per_cu.get ()))
 	{
 	  if (per_cu->v.quick->file_names != nullptr)
 	    qfn_cache.insert (per_cu->v.quick->file_names);
@@ -4430,7 +4431,8 @@
   for (const auto &per_cu : per_objfile->per_bfd->all_comp_units)
     {
       /* We only need to look at symtabs not already expanded.  */
-      if (per_objfile->symtab_set_p (per_cu.get ()))
+      if (per_cu->is_debug_types
+	  || per_objfile->symtab_set_p (per_cu.get ()))
 	continue;
 
       quick_file_names *file_data = dw2_get_file_names (per_cu.get (),