diff mbox series

Fix ICE in output_rnglists, at dwarf2out.c:12294 [PR100515]

Message ID AM8PR10MB470867EB2B44D2CCEBA6EDD0E4529@AM8PR10MB4708.EURPRD10.PROD.OUTLOOK.COM
State New
Headers show
Series Fix ICE in output_rnglists, at dwarf2out.c:12294 [PR100515] | expand

Commit Message

Bernd Edlinger May 12, 2021, 8:11 a.m. UTC 
Hi,

this fixes another regression from my previous patch.


Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
Is it OK for trunk?


Thanks
Bernd.

Comments

Richard Biener May 12, 2021, 8:24 a.m. UTC | #1 
On Wed, 12 May 2021, Bernd Edlinger wrote:

> Hi,
> 
> this fixes another regression from my previous patch.
> 
> 
> Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
> Is it OK for trunk?

OK.

Richard.

> 
> Thanks
> Bernd.
>
Christophe Lyon May 17, 2021, 9:30 a.m. UTC | #2 
On Wed, 12 May 2021 at 10:24, Richard Biener <rguenther@suse.de> wrote:
>
> On Wed, 12 May 2021, Bernd Edlinger wrote:
>
> > Hi,
> >
> > this fixes another regression from my previous patch.
> >
> >
> > Bootstrapped and reg-tested on x86_64-pc-linux-gnu.
> > Is it OK for trunk?
>
> OK.
>
> Richard.
>

Hi,

As the new test uses -fopenmp, it fails on targets that do not support it.

I've committed the attached patch to skip the test in such cases.

Christophe

> >
> > Thanks
> > Bernd.
> >
diff mbox series

Patch

From 62da66525c4b7ac1cfd5cad5b8e690ce928802e5 Mon Sep 17 00:00:00 2001
From: Bernd Edlinger <bernd.edlinger@hotmail.de>
Date: Tue, 11 May 2021 17:55:18 +0200
Subject: [PATCH] Fix ICE in output_rnglists, at dwarf2out.c:12294

In this testcase the compile unit consists of a single
text section with a single embedded DECL_IGNORED_P function.
So we have a kind of multi-range text section here.
To avoid an ICE in output_rnglists we need to make sure
that have_multiple_function_sections is set to true.
This is a regression from
e69ac020372 ("Add line debug info for virtual thunks")

2021-05-11  Bernd Edlinger  <bernd.edlinger@hotmail.de>

	PR debug/100515
	* dwarg2out.c (dwarf2out_finish): Set
	have_multiple_function_sections with multi-range text_section.

	* gcc.dg/debug/dwarf2/pr100515.c: New testcase.
---
 gcc/dwarf2out.c                              |  1 +
 gcc/testsuite/gcc.dg/debug/dwarf2/pr100515.c | 19 +++++++++++++++++++
 2 files changed, 20 insertions(+)
 create mode 100644 gcc/testsuite/gcc.dg/debug/dwarf2/pr100515.c

diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c
index a5ec21a..1593092 100644
--- a/gcc/dwarf2out.c
+++ b/gcc/dwarf2out.c
@@ -31956,6 +31956,7 @@  dwarf2out_finish (const char *filename)
             add_AT_addr (main_comp_unit_die, DW_AT_entry_pc, const0_rtx, true);
 
 	  add_ranges (NULL);
+	  have_multiple_function_sections = true;
 	}
     }
 
diff --git a/gcc/testsuite/gcc.dg/debug/dwarf2/pr100515.c b/gcc/testsuite/gcc.dg/debug/dwarf2/pr100515.c
new file mode 100644
index 0000000..7c72fcd
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/debug/dwarf2/pr100515.c
@@ -0,0 +1,19 @@ 
+/* PR debug/100515 */
+/* { dg-do compile } */
+/* { dg-options "-g -O2 -fopenmp" } */
+
+void
+foo (int x)
+{
+#pragma omp taskloop
+  for (int i = 0; i < x; i++)
+    ;
+}
+
+void
+bar (int x)
+{
+#pragma omp taskloop
+  for (int i = 0; i < x; i++)
+    ;
+}
-- 
1.9.1