]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
debug/113488 - DW_AT_abstract_origin to self
authorRichard Biener <rguenther@suse.de>
Fri, 19 Jan 2024 08:50:43 +0000 (09:50 +0100)
committerRichard Biener <rguenther@suse.de>
Fri, 19 Jan 2024 09:43:07 +0000 (10:43 +0100)
The new sanity check avoiding creating of DIE refs to self triggers
on the PRs testcase when using -g1 and -ffat-lto-objects as while
early DWARF with -g1 doesn't contain any DIEs for LABEL_DECLs later
cloning will still mark DECLs as in if they would via
dwarf2out_abstract_function calling set_block_origin_self.

Instead of messing with the delicate setup of dwarf2out at this stage
the following simply rectifies things after the fact during LTO
streaming when the decl indicates there's an early DIE but there
isn't fixup that indication.

PR debug/113488
* lto-streamer-in.cc (lto_read_tree_1): When there isn't
an early DIE but there should be, do not pretend there is.

gcc/lto-streamer-in.cc

index bef0743b2cd678df32abba76bbd84417853bb9c5..ad0ca24007a08f0dc7192e8232aa9fe5259b29eb 100644 (file)
@@ -1746,6 +1746,11 @@ lto_read_tree_1 (class lto_input_block *ib, class data_in *data_in, tree expr)
          dref_entry e = { expr, str, off };
          dref_queue.safe_push (e);
        }
+      /* When there's no early DIE to refer to but dwarf2out set up
+        things in a way to expect that fixup.  This tends to happen
+        with -g1, see for example PR113488.  */
+      else if (DECL_P (expr) && DECL_ABSTRACT_ORIGIN (expr) == expr)
+       DECL_ABSTRACT_ORIGIN (expr) = NULL_TREE;
     }
 }