From: Jakub Jelinek Date: Sat, 5 Mar 2016 06:50:23 +0000 (+0100) Subject: re PR c++/70084 (va_arg ((ap), int) regression on s390*-*) X-Git-Tag: basepoints/gcc-7~564 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=df762bb20ae6b04ecfb732400e1db6dd55b9f0de;p=thirdparty%2Fgcc.git re PR c++/70084 (va_arg ((ap), int) regression on s390*-*) PR c++/70084 * tree-inline.c (copy_tree_body_r): When cancelling ADDR_EXPR of INDIRECT_REF and ADDR_EXPR changed type, fold_convert it to the right type. * g++.dg/expr/stdarg3.C: New test. From-SVN: r234004 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7d2034bbca2b..1c775b255a6a 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2016-03-05 Jakub Jelinek + + PR c++/70084 + * tree-inline.c (copy_tree_body_r): When cancelling ADDR_EXPR + of INDIRECT_REF and ADDR_EXPR changed type, fold_convert it + to the right type. + 2016-03-04 Bernd Schmidt PR c/69973 diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 4b08dcd7a04f..808b0c8c31fc 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2016-03-05 Jakub Jelinek + + PR c++/70084 + * g++.dg/expr/stdarg3.C: New test. + 2016-03-04 Jeff Law PR tree-optimization/69196 diff --git a/gcc/testsuite/g++.dg/expr/stdarg3.C b/gcc/testsuite/g++.dg/expr/stdarg3.C new file mode 100644 index 000000000000..97aade00e040 --- /dev/null +++ b/gcc/testsuite/g++.dg/expr/stdarg3.C @@ -0,0 +1,18 @@ +// PR c++/70084 +// { dg-do compile } + +#include + +struct A +{ + A (const char *f, ...); +}; + +A::A (const char *f, ...) +{ + va_list ap; + va_start (ap, f); + int i = va_arg (ap, int); // { dg-bogus "first argument to 'va_arg' not of type 'va_list'" } + int j = va_arg ((ap), int); // { dg-bogus "first argument to 'va_arg' not of type 'va_list'" } + va_end (ap); +} diff --git a/gcc/tree-inline.c b/gcc/tree-inline.c index 073c1c78877c..d52e0c6e2027 100644 --- a/gcc/tree-inline.c +++ b/gcc/tree-inline.c @@ -1266,7 +1266,12 @@ copy_tree_body_r (tree *tp, int *walk_subtrees, void *data) /* Handle the case where we substituted an INDIRECT_REF into the operand of the ADDR_EXPR. */ if (TREE_CODE (TREE_OPERAND (*tp, 0)) == INDIRECT_REF) - *tp = TREE_OPERAND (TREE_OPERAND (*tp, 0), 0); + { + tree t = TREE_OPERAND (TREE_OPERAND (*tp, 0), 0); + if (TREE_TYPE (t) != TREE_TYPE (*tp)) + t = fold_convert (remap_type (TREE_TYPE (*tp), id), t); + *tp = t; + } else recompute_tree_invariant_for_addr_expr (*tp);