https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/90195
>From f708694fc2686684589dca7b8f3738a117fc047e Mon Sep 17 00:00:00 2001
From: Younan Zhang <zyn7...@gmail.com>
Date: Fri, 26 Apr 2024 19:06:57 +0800
Subject: [PATCH 1/3] [Sema] Avoid an undesired pack expansion while
transforming PackIndexingType
---
clang/lib/Sema/TreeTransform.h | 3 +++
clang/test/SemaCXX/cxx2c-pack-indexing.cpp | 19 +++++++++++++++++++
2 files changed, 22 insertions(+)
diff --git a/clang/lib/Sema/TreeTransform.h b/clang/lib/Sema/TreeTransform.h
index 9404be5a46f3f7..abc4a16c004a9f 100644
--- a/clang/lib/Sema/TreeTransform.h
+++ b/clang/lib/Sema/TreeTransform.h
@@ -6649,6 +6649,9 @@
TreeTransform<Derived>::TransformPackIndexingType(TypeLocBuilder &TLB,
}
}
+ // We may be doing this in the context of expanding the Pattern. Forget that
+ // because it has been handled above.
+ Sema::ArgumentPackSubstitutionIndexRAII SubstIndex(getSema(), -1);
QualType Result = getDerived().TransformType(TLB, TL.getPatternLoc());
QualType Out = getDerived().RebuildPackIndexingType(
diff --git a/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
b/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
index 606715e6aacffd..2fd0dbfed294a5 100644
--- a/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
+++ b/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
@@ -160,3 +160,22 @@ namespace GH88929 {
using E = P...[0]; // expected-error {{unknown type name 'P'}} \
// expected-error {{expected ';' after alias
declaration}}
}
+
+namespace GH88925 {
+template <typename...> struct S {};
+
+template <int...> struct sequence {};
+
+template <typename... Args, int... indices> auto f(sequence<indices...>) {
+ return S<Args...[indices]...>(); // #use
+}
+
+void g() {
+ static_assert(__is_same(decltype(f<int>(sequence<0, 0>())), S<int, int>));
+ static_assert(__is_same(decltype(f<int, long>(sequence<0, 0>())), S<int,
int>));
+ static_assert(__is_same(decltype(f<int, long>(sequence<0, 1>())), S<int,
long>));
+ f<int, long>(sequence<3>());
+ // expected-error@#use {{invalid index 3 for pack 'Args' of size 2}}}
+ // expected-note-re@-2 {{function template specialization '{{.*}}' requested
here}}
+}
+}
>From 7b0ae16b6777c6e98df64cd2366434972fc68164 Mon Sep 17 00:00:00 2001
From: Younan Zhang <zyn7...@gmail.com>
Date: Sat, 27 Apr 2024 12:07:30 +0800
Subject: [PATCH 2/3] Clarify comments
---
clang/lib/Sema/TreeTransform.h | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/clang/lib/Sema/TreeTransform.h b/clang/lib/Sema/TreeTransform.h
index abc4a16c004a9f..b50fdab8bfd05e 100644
--- a/clang/lib/Sema/TreeTransform.h
+++ b/clang/lib/Sema/TreeTransform.h
@@ -6649,8 +6649,9 @@
TreeTransform<Derived>::TransformPackIndexingType(TypeLocBuilder &TLB,
}
}
- // We may be doing this in the context of expanding the Pattern. Forget that
- // because it has been handled above.
+ // A pack indexing type can appear in a larger pack expansion,
+ // e.g. `Pack...[pack_of_indexes]...`
+ // so we need to temporarily disable substitution of pack elements
Sema::ArgumentPackSubstitutionIndexRAII SubstIndex(getSema(), -1);
QualType Result = getDerived().TransformType(TLB, TL.getPatternLoc());
>From d4a2f1266114b2bc96c17f4d0065338bb0040fb1 Mon Sep 17 00:00:00 2001
From: Younan Zhang <zyn7...@gmail.com>
Date: Sun, 28 Apr 2024 13:29:37 +0800
Subject: [PATCH 3/3] Make dependnt PackIndexingExpr always an LValue
---
clang/lib/AST/ExprClassification.cpp | 8 +++++++-
clang/test/SemaCXX/cxx2c-pack-indexing.cpp | 23 ++++++++++++++++++----
2 files changed, 26 insertions(+), 5 deletions(-)
diff --git a/clang/lib/AST/ExprClassification.cpp
b/clang/lib/AST/ExprClassification.cpp
index 7026fca8554ce9..1c4be91b0ce80f 100644
--- a/clang/lib/AST/ExprClassification.cpp
+++ b/clang/lib/AST/ExprClassification.cpp
@@ -216,8 +216,14 @@ static Cl::Kinds ClassifyInternal(ASTContext &Ctx, const
Expr *E) {
return ClassifyInternal(Ctx,
cast<SubstNonTypeTemplateParmExpr>(E)->getReplacement());
- case Expr::PackIndexingExprClass:
+ case Expr::PackIndexingExprClass: {
+ // A dependent pack-index-expression now is supposed to denote a function
+ // parameter pack, an NTTP pack, or the pack introduced by a structured
+ // binding. Consider it as an LValue expression.
+ if (cast<PackIndexingExpr>(E)->isInstantiationDependent())
+ return Cl::CL_LValue;
return ClassifyInternal(Ctx, cast<PackIndexingExpr>(E)->getSelectedExpr());
+ }
// C, C++98 [expr.sub]p1: The result is an lvalue of type "T".
// C++11 (DR1213): in the case of an array operand, the result is an lvalue
diff --git a/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
b/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
index 2fd0dbfed294a5..a3e5a0931491b5 100644
--- a/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
+++ b/clang/test/SemaCXX/cxx2c-pack-indexing.cpp
@@ -164,18 +164,33 @@ namespace GH88929 {
namespace GH88925 {
template <typename...> struct S {};
+template <auto...> struct W {};
+
template <int...> struct sequence {};
-template <typename... Args, int... indices> auto f(sequence<indices...>) {
- return S<Args...[indices]...>(); // #use
+template <typename... args, int... indices> auto f(sequence<indices...>) {
+ return S<args...[indices]...>(); // #use
}
-void g() {
+template <auto... args, int... indices> auto g(sequence<indices...>) {
+ return W<args...[indices]...>(); // #nttp-use
+}
+
+void h() {
static_assert(__is_same(decltype(f<int>(sequence<0, 0>())), S<int, int>));
static_assert(__is_same(decltype(f<int, long>(sequence<0, 0>())), S<int,
int>));
static_assert(__is_same(decltype(f<int, long>(sequence<0, 1>())), S<int,
long>));
f<int, long>(sequence<3>());
- // expected-error@#use {{invalid index 3 for pack 'Args' of size 2}}}
+ // expected-error@#use {{invalid index 3 for pack 'args' of size 2}}}
+ // expected-note-re@-2 {{function template specialization '{{.*}}' requested
here}}
+
+ struct foo {};
+ struct bar {};
+ struct baz {};
+
+ static_assert(__is_same(decltype(g<foo{}, bar{}, baz{}>(sequence<0, 2,
1>())), W<foo{}, baz{}, bar{}>));
+ g<foo{}>(sequence<4>());
+ // expected-error@#nttp-use {{invalid index 4 for pack args of size 1}}
// expected-note-re@-2 {{function template specialization '{{.*}}' requested
here}}
}
}
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
