[C++ Patch] PR 59628

[C++ Patch] PR 59628

[Paolo Carlini]

[-- Attachment #1: Type: text/plain, Size: 208 bytes --]

Hi,

I think that to avoid ICE-ing during error recovery when DECL_SAVED_TREE 
is NULL_TREE we want to simply bail out and return true to the caller. 
Tested x86_64-linux.

Thanks,
Paolo.

//////////////////

[-- Attachment #2: CL_59628 --]
[-- Type: text/plain, Size: 288 bytes --]

/cp
2014-12-12  Paolo Carlini  <paolo.carlini@oracle.com>

	PR c++/59628
	* semantics.c (finish_omp_reduction_clause): Early return true
	if DECL_SAVED_TREE (id) is NULL_TREE.

/testsuite
2014-12-12  Paolo Carlini  <paolo.carlini@oracle.com>

	PR c++/59628
	* g++.dg/gomp/pr59628.C: New.

[-- Attachment #3: patch_59628 --]
[-- Type: text/plain, Size: 922 bytes --]

Index: cp/semantics.c
===================================================================
--- cp/semantics.c	(revision 218683)
+++ cp/semantics.c	(working copy)
@@ -5138,6 +5138,8 @@ finish_omp_reduction_clause (tree c, bool *need_de
       id = OVL_CURRENT (id);
       mark_used (id);
       tree body = DECL_SAVED_TREE (id);
+      if (!body)
+	return true;
       if (TREE_CODE (body) == STATEMENT_LIST)
 	{
 	  tree_stmt_iterator tsi;
Index: testsuite/g++.dg/gomp/pr59628.C
===================================================================
--- testsuite/g++.dg/gomp/pr59628.C	(revision 0)
+++ testsuite/g++.dg/gomp/pr59628.C	(working copy)
@@ -0,0 +1,13 @@
+// PR c++/59628
+// { dg-do compile }
+// { dg-options "-fopenmp" }
+
+struct A { int i; };
+
+void foo()
+{
+  A a;
+  #pragma omp declare reduction (+: A: omp_out.i +: omp_in.i)  // { dg-error "expected" }
+  #pragma omp parallel reduction (+: a)
+  ;
+}

Re: [C++ Patch] PR 59628

[Jakub Jelinek]

On Fri, Dec 12, 2014 at 10:13:39PM +0100, Paolo Carlini wrote:
> Hi,
> 
> I think that to avoid ICE-ing during error recovery when DECL_SAVED_TREE is
> NULL_TREE we want to simply bail out and return true to the caller. Tested
> x86_64-linux.
> 
> Thanks,
> Paolo.
> 
> //////////////////

> /cp
> 2014-12-12  Paolo Carlini  <paolo.carlini@oracle.com>
> 
> 	PR c++/59628
> 	* semantics.c (finish_omp_reduction_clause): Early return true
> 	if DECL_SAVED_TREE (id) is NULL_TREE.
> 
> /testsuite
> 2014-12-12  Paolo Carlini  <paolo.carlini@oracle.com>
> 
> 	PR c++/59628
> 	* g++.dg/gomp/pr59628.C: New.

Ok, thanks.

	Jakub