c++: Add TARGET_EXPR comments
authorJason Merrill <jason@redhat.com>
Mon, 4 Jan 2021 21:11:08 +0000 (16:11 -0500)
committerJason Merrill <jason@redhat.com>
Thu, 7 Jan 2021 21:09:11 +0000 (16:09 -0500)
Discussing the 98469 patch and class prvalues with Jakub led me to
double-check our handling of TARGET_EXPR in constexpr.c, and add a note
about why we don't strip them in parameter initialization.  And another to
clarify that we're handling an INIT_EXPR in a place we do strip them.

gcc/cp/ChangeLog:

* constexpr.c (cxx_bind_parameters_in_call): Add comment.
(cxx_eval_store_expression): Add comment.

gcc/cp/constexpr.c

index 0c12f608d36cc02c8c0be84bda86572f8ae13633..4a5e6384da68661d96878788528c696e512194c9 100644 (file)
@@ -1591,6 +1591,9 @@ cxx_bind_parameters_in_call (const constexpr_ctx *ctx, tree t,
       if (TREE_ADDRESSABLE (type))
        /* Undo convert_for_arg_passing work here.  */
        x = convert_from_reference (x);
+      /* Normally we would strip a TARGET_EXPR in an initialization context
+        such as this, but here we do the elision differently: we keep the
+        TARGET_EXPR, and use its CONSTRUCTOR as the value of the parm.  */
       arg = cxx_eval_constant_expression (ctx, x, /*lval=*/false,
                                          non_constant_p, overflow_p);
       /* Don't VERIFY_CONSTANT here.  */
@@ -5388,6 +5391,9 @@ cxx_eval_store_expression (const constexpr_ctx *ctx, tree t,
 
   if (!preeval)
     {
+      /* We're handling an INIT_EXPR of class type, so the value of the
+        initializer can depend on the object it's initializing.  */
+
       /* Create a new CONSTRUCTOR in case evaluation of the initializer
         wants to modify it.  */
       if (*valp == NULL_TREE)