diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index 44788808193..52cd542bb97 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,9 @@ +2017-11-07 Jakub Jelinek + + PR c++/82835 + * cp-gimplify.c (cxx_omp_clause_apply_fn): For methods pass i - 1 to + convert_default_arg instead of i. + 2017-11-06 Jason Merrill P0704R1 - fixing const-qualified pointers to members diff --git a/gcc/cp/cp-gimplify.c b/gcc/cp/cp-gimplify.c index 014c1ee7231..7c7c0409af8 100644 --- a/gcc/cp/cp-gimplify.c +++ b/gcc/cp/cp-gimplify.c @@ -1717,6 +1717,7 @@ cxx_omp_clause_apply_fn (tree fn, tree arg1, tree arg2) if (arg2) defparm = TREE_CHAIN (defparm); + bool is_method = TREE_CODE (TREE_TYPE (fn)) == METHOD_TYPE; if (TREE_CODE (TREE_TYPE (arg1)) == ARRAY_TYPE) { tree inner_type = TREE_TYPE (arg1); @@ -1765,8 +1766,8 @@ cxx_omp_clause_apply_fn (tree fn, tree arg1, tree arg2) 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, - tf_warning_or_error); + TREE_PURPOSE (parm), fn, + i - is_method, tf_warning_or_error); t = build_call_a (fn, i, argarray); t = fold_convert (void_type_node, t); t = fold_build_cleanup_point_expr (TREE_TYPE (t), t); @@ -1798,8 +1799,8 @@ cxx_omp_clause_apply_fn (tree fn, tree arg1, tree arg2) 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, tf_warning_or_error); + TREE_PURPOSE (parm), fn, + i - is_method, tf_warning_or_error); t = build_call_a (fn, i, argarray); t = fold_convert (void_type_node, t); return fold_build_cleanup_point_expr (TREE_TYPE (t), t); diff --git a/libgomp/ChangeLog b/libgomp/ChangeLog index 7c0e808d587..19966bd0947 100644 --- a/libgomp/ChangeLog +++ b/libgomp/ChangeLog @@ -1,3 +1,8 @@ +2017-11-07 Jakub Jelinek + + PR c++/82835 + * testsuite/libgomp.c++/pr82835.C: New test. + 2017-11-06 Martin Liska * testsuite/libgomp.c++/loop-2.C: Return a value diff --git a/libgomp/testsuite/libgomp.c++/pr82835.C b/libgomp/testsuite/libgomp.c++/pr82835.C new file mode 100644 index 00000000000..df64ecfb1cf --- /dev/null +++ b/libgomp/testsuite/libgomp.c++/pr82835.C @@ -0,0 +1,34 @@ +// PR c++/82835 +// { dg-do run } + +int a, b; + +template +struct C { + C (int x = a) : c (5) { if (x != 137) __builtin_abort (); } + int c; +}; + +struct D { + void foo (); + int d; +}; + +void +D::foo () +{ + C c; +#pragma omp for private (c) + for (b = 0; b < d; b++) + c.c++; +} + +int +main () +{ + a = 137; + D d; + d.d = 16; + d.foo (); + return 0; +}