[clang] [Clang] Static member initializers are not immediate escalating context. (PR #66021)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 13 01:17:39 PDT 2023
https://github.com/cor3ntin updated https://github.com/llvm/llvm-project/pull/66021:
>From a3390c4ac8bba43e73ea41f4af85b37df97b1098 Mon Sep 17 00:00:00 2001
From: Corentin Jabot <corentinjabot at gmail.com>
Date: Mon, 11 Sep 2023 23:40:38 +0200
Subject: [PATCH 1/2] [Clang] Static member initializers are not immediate
escalating context.
Per CWG2760, default members initializers should be consider
part the body of consstructors, which mean they are evaluated
in an immediate escalating context.
However, this does not apply to static members.
This patch produces some extraneous diagnostics, unfortunately
we do not have a good way to report an error back to the
initializer and this is a preexisting issue
Fixes #65985
---
clang/docs/ReleaseNotes.rst | 4 ++++
clang/lib/Parse/ParseDeclCXX.cpp | 12 ++++++++--
.../SemaCXX/cxx2b-consteval-propagate.cpp | 23 +++++++++++++++++++
3 files changed, 37 insertions(+), 2 deletions(-)
diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index 3cdad2f7b9f0e5a..8c8ec0e11e3dd7d 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -281,6 +281,10 @@ Bug Fixes to C++ Support
a non-template inner-class between the function and the class template.
(`#65810 <https://github.com/llvm/llvm-project/issues/65810>`_)
+- Fix a crash when calling a non-constant immediate function
+ in the initializer of a static data member.
+ (`#65985 <https://github.com/llvm/llvm-project/issues/65985>_`).
+
Bug Fixes to AST Handling
^^^^^^^^^^^^^^^^^^^^^^^^^
- Fixed an import failure of recursive friend class template.
diff --git a/clang/lib/Parse/ParseDeclCXX.cpp b/clang/lib/Parse/ParseDeclCXX.cpp
index 730b6e55246d6b7..b4a72d599da6655 100644
--- a/clang/lib/Parse/ParseDeclCXX.cpp
+++ b/clang/lib/Parse/ParseDeclCXX.cpp
@@ -3230,13 +3230,21 @@ ExprResult Parser::ParseCXXMemberInitializer(Decl *D, bool IsFunction,
assert(Tok.isOneOf(tok::equal, tok::l_brace) &&
"Data member initializer not starting with '=' or '{'");
+ bool IsFieldInitialization = isa_and_present<FieldDecl>(D);
+
EnterExpressionEvaluationContext Context(
Actions,
- isa_and_present<FieldDecl>(D)
+ IsFieldInitialization
? Sema::ExpressionEvaluationContext::PotentiallyEvaluatedIfUsed
: Sema::ExpressionEvaluationContext::PotentiallyEvaluated,
D);
- Actions.ExprEvalContexts.back().InImmediateEscalatingFunctionContext = true;
+
+ // CWG2760
+ // Default member initializers used to initialize a base or member subobject
+ // [...] are considered to be part of the function body
+ Actions.ExprEvalContexts.back().InImmediateEscalatingFunctionContext =
+ IsFieldInitialization;
+
if (TryConsumeToken(tok::equal, EqualLoc)) {
if (Tok.is(tok::kw_delete)) {
// In principle, an initializer of '= delete p;' is legal, but it will
diff --git a/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp b/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp
index c5eb7f139327505..f967ce6554d777c 100644
--- a/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp
+++ b/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp
@@ -330,3 +330,26 @@ struct S {
S s(0); // expected-note {{in the default initializer of 'j'}}
}
+
+namespace GH65985 {
+
+int consteval operator""_foo(unsigned long long V) {
+ return 0;
+}
+int consteval operator""_bar(unsigned long long V); // expected-note 3{{here}}
+
+struct C {
+ static const int a = 1_foo;
+ static constexpr int b = 1_foo;
+ static const int c = 1_bar; // expected-error {{call to consteval function 'GH65985::operator""_bar' is not a constant expression}} \
+ // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}} \
+ // expected-error {{in-class initializer for static data member is not a constant expression}}
+
+ // FIXME: remove duplicate diagnostics
+ static constexpr int d = 1_bar; // expected-error {{call to consteval function 'GH65985::operator""_bar' is not a constant expression}} \
+ // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}} \
+ // expected-error {{constexpr variable 'd' must be initialized by a constant expression}} \
+ // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}}
+};
+
+}
>From 12e04c8bd51d1f4e0d77969642f115b554b3a589 Mon Sep 17 00:00:00 2001
From: Corentin Jabot <corentinjabot at gmail.com>
Date: Wed, 13 Sep 2023 10:10:28 +0200
Subject: [PATCH 2/2] Add more tests
---
clang/test/SemaCXX/cxx2a-consteval.cpp | 35 +++++++++++++++++++
.../SemaCXX/cxx2b-consteval-propagate.cpp | 23 ------------
2 files changed, 35 insertions(+), 23 deletions(-)
diff --git a/clang/test/SemaCXX/cxx2a-consteval.cpp b/clang/test/SemaCXX/cxx2a-consteval.cpp
index d98ec8048c32466..82d3212ed8156b4 100644
--- a/clang/test/SemaCXX/cxx2a-consteval.cpp
+++ b/clang/test/SemaCXX/cxx2a-consteval.cpp
@@ -1126,4 +1126,39 @@ int test2() { return h{nullptr}; }
// expected-note at -2 {{subobject 'g' is not initialized}}
+}
+
+namespace GH65985 {
+
+int consteval operator""_foo(unsigned long long V) {
+ return 0;
+}
+int consteval operator""_bar(unsigned long long V); // expected-note 3{{here}}
+
+int consteval f() {
+ return 0;
+}
+
+int consteval g(); // expected-note {{here}}
+
+
+struct C {
+ static const int a = 1_foo;
+ static constexpr int b = 1_foo;
+ static const int c = 1_bar; // expected-error {{call to consteval function 'GH65985::operator""_bar' is not a constant expression}} \
+ // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}} \
+ // expected-error {{in-class initializer for static data member is not a constant expression}}
+
+ // FIXME: remove duplicate diagnostics
+ static constexpr int d = 1_bar; // expected-error {{call to consteval function 'GH65985::operator""_bar' is not a constant expression}} \
+ // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}} \
+ // expected-error {{constexpr variable 'd' must be initialized by a constant expression}} \
+ // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}}
+
+ static const int e = f();
+ static const int f = g(); // expected-error {{call to consteval function 'GH65985::g' is not a constant expression}} \
+ // expected-error {{in-class initializer for static data member is not a constant expression}} \
+ // expected-note {{undefined function 'g' cannot be used in a constant expression}}
+};
+
}
diff --git a/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp b/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp
index f967ce6554d777c..c5eb7f139327505 100644
--- a/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp
+++ b/clang/test/SemaCXX/cxx2b-consteval-propagate.cpp
@@ -330,26 +330,3 @@ struct S {
S s(0); // expected-note {{in the default initializer of 'j'}}
}
-
-namespace GH65985 {
-
-int consteval operator""_foo(unsigned long long V) {
- return 0;
-}
-int consteval operator""_bar(unsigned long long V); // expected-note 3{{here}}
-
-struct C {
- static const int a = 1_foo;
- static constexpr int b = 1_foo;
- static const int c = 1_bar; // expected-error {{call to consteval function 'GH65985::operator""_bar' is not a constant expression}} \
- // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}} \
- // expected-error {{in-class initializer for static data member is not a constant expression}}
-
- // FIXME: remove duplicate diagnostics
- static constexpr int d = 1_bar; // expected-error {{call to consteval function 'GH65985::operator""_bar' is not a constant expression}} \
- // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}} \
- // expected-error {{constexpr variable 'd' must be initialized by a constant expression}} \
- // expected-note {{undefined function 'operator""_bar' cannot be used in a constant expression}}
-};
-
-}
More information about the cfe-commits
mailing list