PR c++/33799 - destroy return value if local cleanup throws.
authorJason Merrill <jason@redhat.com>
Fri, 10 Jan 2020 22:14:12 +0000 (17:14 -0500)
committerJason Merrill <jason@redhat.com>
Mon, 13 Jan 2020 17:50:12 +0000 (12:50 -0500)
commit7c82dd6c02d44d9d2cd84dda137c00b1a3cd6c90
treed5bf3e04c5a748eb795b36dd2aa133add540fa74
parentf1acad4e43908e90ca2b5155a878639cbea4c4e1
PR c++/33799 - destroy return value if local cleanup throws.

This is a pretty rare situation since the C++11 change to make all
destructors default to noexcept, but it is still possible to define throwing
destructors, and if a destructor for a local variable throws during the
return, we've already constructed the return value, so now we need to
destroy it.  I handled this somewhat like the new-expression cleanup; as in
that case, this cleanup can't properly nest with the cleanups for local
variables, so I introduce a cleanup region around the whole function and a
flag variable to indicate whether the return value actually needs to be
destroyed.

Setting the flag requires giving a COMPOUND_EXPR as the operand of a
RETURN_EXPR, so I adjust gimplify_return_expr to handle that.

This doesn't currently work with deduced return type because we don't know
the type when we're deciding whether to introduce the cleanup region.

gcc/
* gimplify.c (gimplify_return_expr): Handle COMPOUND_EXPR.
gcc/cp/
* cp-tree.h (current_retval_sentinel): New macro.
* decl.c (start_preparsed_function): Set up cleanup for retval.
* typeck.c (check_return_expr): Set current_retval_sentinel.
gcc/ChangeLog
gcc/cp/ChangeLog
gcc/cp/cp-tree.h
gcc/cp/decl.c
gcc/cp/typeck.c
gcc/gimplify.c
gcc/testsuite/g++.dg/eh/return1.C [new file with mode: 0644]