From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <jason@sourceware.org>
Received: by sourceware.org (Postfix, from userid 2122)
 id 71AB63858407; Mon, 28 Mar 2022 19:12:57 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 71AB63858407
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="utf-8"
From: Jason Merrill <jason@gcc.gnu.org>
To: gcc-cvs@gcc.gnu.org
Subject: [gcc r11-9704] c++: member alias declaration [PR103968]
X-Act-Checkin: gcc
X-Git-Author: Jason Merrill <jason@redhat.com>
X-Git-Refname: refs/heads/releases/gcc-11
X-Git-Oldrev: e952290874d6b99946dc02e125c0fb0e9b13a1e3
X-Git-Newrev: 97390a9914672f5ce73c169b3ae7fd4bba25c2fe
Message-Id: <20220328191257.71AB63858407@sourceware.org>
Date: Mon, 28 Mar 2022 19:12:57 +0000 (GMT)
X-BeenThere: gcc-cvs@gcc.gnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Gcc-cvs mailing list <gcc-cvs.gcc.gnu.org>
List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-cvs>,
 <mailto:gcc-cvs-request@gcc.gnu.org?subject=unsubscribe>
List-Archive: <https://gcc.gnu.org/pipermail/gcc-cvs/>
List-Help: <mailto:gcc-cvs-request@gcc.gnu.org?subject=help>
List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-cvs>,
 <mailto:gcc-cvs-request@gcc.gnu.org?subject=subscribe>
X-List-Received-Date: Mon, 28 Mar 2022 19:12:57 -0000

https://gcc.gnu.org/g:97390a9914672f5ce73c169b3ae7fd4bba25c2fe

commit r11-9704-g97390a9914672f5ce73c169b3ae7fd4bba25c2fe
Author: Jason Merrill <jason@redhat.com>
Date:   Sun Mar 27 00:28:30 2022 -0400

    c++: member alias declaration [PR103968]
    
    Here, we were wrongly thinking that (const Options&)Widget<T>::c_options is
    not value-dependent because neither the type nor the (value of) c_options
    are dependent, but since we're binding it to a reference we also need to
    consider that it has a value-dependent address.
    
            PR c++/103968
    
    gcc/cp/ChangeLog:
    
            * pt.c (value_dependent_expression_p): Check
            has_value_dependent_address for conversion to reference.
    
    gcc/testsuite/ChangeLog:
    
            * g++.dg/cpp0x/alias-decl-mem1.C: New test.

Diff:
---
 gcc/cp/pt.c                                  |  4 ++++
 gcc/testsuite/g++.dg/cpp0x/alias-decl-mem1.C | 20 ++++++++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/gcc/cp/pt.c b/gcc/cp/pt.c
index 1493019bd27..b80d3d13600 100644
--- a/gcc/cp/pt.c
+++ b/gcc/cp/pt.c
@@ -26987,6 +26987,10 @@ value_dependent_expression_p (tree expression)
 	if (TREE_CODE (expression) == TREE_LIST)
 	  return any_value_dependent_elements_p (expression);
 
+	if (TREE_CODE (type) == REFERENCE_TYPE
+	    && has_value_dependent_address (expression))
+	  return true;
+
 	return value_dependent_expression_p (expression);
       }
 
diff --git a/gcc/testsuite/g++.dg/cpp0x/alias-decl-mem1.C b/gcc/testsuite/g++.dg/cpp0x/alias-decl-mem1.C
new file mode 100644
index 00000000000..3e422de4054
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/alias-decl-mem1.C
@@ -0,0 +1,20 @@
+// PR c++/103968
+// { dg-do compile { target c++11 } }
+
+template <typename Opt, Opt const& options>
+struct trait
+{
+    template <typename T>
+    struct NonInstantiated{};
+};
+
+struct Options {};
+
+template <typename T>
+struct Widget
+{
+    static constexpr auto c_options = Options{};
+    using Trait = trait<decltype(c_options), c_options>;
+};
+
+Widget<int>::Trait b{}; // Crashes GCC > 10.3