[gcc r12-7398] c++: (*(fn))() [PR104618]

[Jason Merrill <jason@gcc.gnu.org>]

https://gcc.gnu.org/g:0096b0b467b2c98ed258f0ead9a8b87b2a6d8466

commit r12-7398-g0096b0b467b2c98ed258f0ead9a8b87b2a6d8466
Author: Jason Merrill <jason@redhat.com>
Date:   Fri Feb 25 15:26:17 2022 -0400

    c++: (*(fn))() [PR104618]
    
    The patch for PR90451 deferred marking to the point of actual use; we missed
    this one because of the parens.
    
            PR c++/104618
    
    gcc/cp/ChangeLog:
    
            * typeck.cc (cp_build_addr_expr_1): Also
            maybe_undo_parenthesized_ref.
    
    gcc/testsuite/ChangeLog:
    
            * g++.dg/overload/paren1.C: New test.

Diff:
---
 gcc/cp/typeck.cc                       | 6 +++---
 gcc/testsuite/g++.dg/overload/paren1.C | 7 +++++++
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/gcc/cp/typeck.cc b/gcc/cp/typeck.cc
index f796337f73c..bddc83759ad 100644
--- a/gcc/cp/typeck.cc
+++ b/gcc/cp/typeck.cc
@@ -6884,9 +6884,9 @@ cp_build_addr_expr_1 (tree arg, bool strict_lvalue, tsubst_flags_t complain)
      so we can just form an ADDR_EXPR with the correct type.  */
   if (processing_template_decl || TREE_CODE (arg) != COMPONENT_REF)
     {
-      tree stripped_arg = tree_strip_any_location_wrapper (arg);
-      if (TREE_CODE (stripped_arg) == FUNCTION_DECL
-	  && !mark_used (stripped_arg, complain) && !(complain & tf_error))
+      tree stripped_arg
+	= tree_strip_any_location_wrapper (maybe_undo_parenthesized_ref (arg));
+      if (!mark_single_function (stripped_arg, complain))
 	return error_mark_node;
       val = build_address (arg);
       if (TREE_CODE (arg) == OFFSET_REF)
diff --git a/gcc/testsuite/g++.dg/overload/paren1.C b/gcc/testsuite/g++.dg/overload/paren1.C
new file mode 100644
index 00000000000..21421ae8726
--- /dev/null
+++ b/gcc/testsuite/g++.dg/overload/paren1.C
@@ -0,0 +1,7 @@
+// PR c++/104618
+
+extern void gen_addsi3 (void);
+void output_stack_adjust ()
+{
+  (*(gen_addsi3)) ();
+}