diff options
| author | Jakub Jelinek <jakub@redhat.com> | 2008-02-19 11:18:29 +0100 |
|---|---|---|
| committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2008-02-19 11:18:29 +0100 |
| commit | d2ee546fd6d04c56754647e5c21d3fa7e4f11294 (patch) | |
| tree | f5f6359a8ad0fd37c584c6e4a04552ab5d730271 | |
| parent | edb6000e3720a4944ab375e01a933f658c00d4b0 (diff) | |
| download | gcc-d2ee546fd6d04c56754647e5c21d3fa7e4f11294.tar.gz | |
re PR c++/35028 (ICE with strange ctor and firstprivate)
PR c++/35028
* cp-gimplify.c (cxx_omp_clause_apply_fn): Handle vararg copy ctors.
* g++.dg/gomp/pr35028.C: New test.
From-SVN: r132426
| -rw-r--r-- | gcc/cp/ChangeLog | 3 | ||||
| -rw-r--r-- | gcc/cp/cp-gimplify.c | 5 | ||||
| -rw-r--r-- | gcc/testsuite/ChangeLog | 3 | ||||
| -rw-r--r-- | gcc/testsuite/g++.dg/gomp/pr35028.C | 19 |
4 files changed, 28 insertions, 2 deletions
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 386ff722c53..6f016fcfce5 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,5 +1,8 @@ 2008-02-19 Jakub Jelinek <jakub@redhat.com> + PR c++/35028 + * cp-gimplify.c (cxx_omp_clause_apply_fn): Handle vararg copy ctors. + PR c++/34964 PR c++/35244 * semantics.c (finish_omp_threadprivate): Do nothing for error_operand_p diff --git a/gcc/cp/cp-gimplify.c b/gcc/cp/cp-gimplify.c index 50b40484413..498ecc5379a 100644 --- a/gcc/cp/cp-gimplify.c +++ b/gcc/cp/cp-gimplify.c @@ -844,7 +844,8 @@ cxx_omp_clause_apply_fn (tree fn, tree arg1, tree arg2) if (arg2) argarray[i++] = p2; /* Handle default arguments. */ - for (parm = defparm; parm != void_list_node; parm = TREE_CHAIN (parm), i++) + for (parm = defparm; parm && parm != void_list_node; + parm = TREE_CHAIN (parm), i++) argarray[i] = convert_default_arg (TREE_VALUE (parm), TREE_PURPOSE (parm), fn, i); t = build_call_a (fn, i, argarray); @@ -875,7 +876,7 @@ cxx_omp_clause_apply_fn (tree fn, tree arg1, tree arg2) if (arg2) argarray[i++] = build_fold_addr_expr (arg2); /* Handle default arguments. */ - for (parm = defparm; parm != void_list_node; + for (parm = defparm; parm && parm != void_list_node; parm = TREE_CHAIN (parm), i++) argarray[i] = convert_default_arg (TREE_VALUE (parm), TREE_PURPOSE (parm), diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index b5ab00f0b67..5e2d40c1d69 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,5 +1,8 @@ 2008-02-19 Jakub Jelinek <jakub@redhat.com> + PR c++/35028 + * g++.dg/gomp/pr35028.C: New test. + PR c++/34964 PR c++/35244 * gcc.dg/gomp/pr34964.c: New test. diff --git a/gcc/testsuite/g++.dg/gomp/pr35028.C b/gcc/testsuite/g++.dg/gomp/pr35028.C new file mode 100644 index 00000000000..ef7ba07de17 --- /dev/null +++ b/gcc/testsuite/g++.dg/gomp/pr35028.C @@ -0,0 +1,19 @@ +// PR c++/35028 +// { dg-do compile } +// { dg-options "-fopenmp" } + +struct A +{ + A (); + A (const A &, ...); + ~A (); + A operator++ (int); +}; + +void +foo () +{ + A a; + #pragma omp parallel firstprivate (a) + a++; +} |