LYP951018 wrote:
I have a draft which implements the capture rule for deferred lambda
instantiation https://github.com/llvm/llvm-project/pull/105953
I've been too busy lately and haven't continued working on this Pull Request.If
you're interested,you can take a look at it.
https://github.com/l
LYP951018 wrote:
Thanks for the review comments. Sorry for the delayed response, I'm a little
busy recently. I'm still fixing a clang-tidy unit test broken by this change.
I'll take a deep dive into your suggestions a bit later ;)
https://github.com/llvm/llvm-project/pull/105953
__
https://github.com/LYP951018 edited
https://github.com/llvm/llvm-project/pull/105953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 edited
https://github.com/llvm/llvm-project/pull/105953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 edited
https://github.com/llvm/llvm-project/pull/105953
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 created
https://github.com/llvm/llvm-project/pull/105953
## Code Changes:
- Currently, Clang requires instantiation to determine captures of a packs
(refer to 7bf8f6fa8ab123fe97ccd82d9a0ddff85505ee5f), but P0588 allows delayed
instantiation of `if constexpr` stat
LYP951018 wrote:
Thanks for the review ;)
https://github.com/llvm/llvm-project/pull/100766
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 closed
https://github.com/llvm/llvm-project/pull/100766
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -712,22 +712,30 @@ bool Sema::addInstantiatedCapturesToScope(
auto AddSingleCapture = [&](const ValueDecl *CapturedPattern,
unsigned Index) {
ValueDecl *CapturedVar = LambdaClass->getCapture(Index)->getCapturedVar();
-if (CapturedVar->
LYP951018 wrote:
ping~
https://github.com/llvm/llvm-project/pull/100766
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 edited
https://github.com/llvm/llvm-project/pull/100766
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 edited
https://github.com/llvm/llvm-project/pull/100766
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 updated
https://github.com/llvm/llvm-project/pull/100766
>From bae8f7c203573bf75e6838551e5203b3ef8d01b8 Mon Sep 17 00:00:00 2001
From: letrec
Date: Fri, 26 Jul 2024 22:25:10 +0800
Subject: [PATCH 1/3] add init capture packs to requires scope
---
clang/include/clan
https://github.com/LYP951018 updated
https://github.com/llvm/llvm-project/pull/100766
>From bae8f7c203573bf75e6838551e5203b3ef8d01b8 Mon Sep 17 00:00:00 2001
From: letrec
Date: Fri, 26 Jul 2024 22:25:10 +0800
Subject: [PATCH 1/2] add init capture packs to requires scope
---
clang/include/clan
LYP951018 wrote:
Thanks for the review ;)
https://github.com/llvm/llvm-project/pull/97215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 closed
https://github.com/llvm/llvm-project/pull/97215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2379,23 +2379,32 @@ Sema::LambdaScopeForCallOperatorInstantiationRAII::
SemaRef.RebuildLambdaScopeInfo(cast(FD));
- FunctionDecl *Pattern = getPatternFunctionDecl(FD);
- if (Pattern) {
-SemaRef.addInstantiatedCapturesToScope(FD, Pattern, Scope, MLTAL);
+ Function
https://github.com/LYP951018 updated
https://github.com/llvm/llvm-project/pull/97215
>From a997ae70f86230200b1082c9bdc0bdf56e30b7c4 Mon Sep 17 00:00:00 2001
From: letrec
Date: Sun, 30 Jun 2024 21:03:23 +0800
Subject: [PATCH 1/2] Fix the order of addInstantiatedParameters in
LambdaScopeForCallO
https://github.com/LYP951018 edited
https://github.com/llvm/llvm-project/pull/97215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 updated
https://github.com/llvm/llvm-project/pull/97215
>From a997ae70f86230200b1082c9bdc0bdf56e30b7c4 Mon Sep 17 00:00:00 2001
From: letrec
Date: Sun, 30 Jun 2024 21:03:23 +0800
Subject: [PATCH] Fix the order of addInstantiatedParameters in
LambdaScopeForCallOpera
LYP951018 wrote:
Sorry for disturbing... I'm trying to get the CI working. I can't figure out
why buildkite/github-pull-requests failed...
https://github.com/llvm/llvm-project/pull/97215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
https://github.com/LYP951018 reopened
https://github.com/llvm/llvm-project/pull/97215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 closed
https://github.com/llvm/llvm-project/pull/97215
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/LYP951018 updated
https://github.com/llvm/llvm-project/pull/97215
>From a997ae70f86230200b1082c9bdc0bdf56e30b7c4 Mon Sep 17 00:00:00 2001
From: letrec
Date: Sun, 30 Jun 2024 21:03:23 +0800
Subject: [PATCH] Fix the order of addInstantiatedParameters in
LambdaScopeForCallOpera
https://github.com/LYP951018 updated
https://github.com/llvm/llvm-project/pull/97215
>From a997ae70f86230200b1082c9bdc0bdf56e30b7c4 Mon Sep 17 00:00:00 2001
From: letrec
Date: Sun, 30 Jun 2024 21:03:23 +0800
Subject: [PATCH] Fix the order of addInstantiatedParameters in
LambdaScopeForCallOpera
https://github.com/LYP951018 created
https://github.com/llvm/llvm-project/pull/97215
Currently, `addInstantiatedParameters` is called from the innermost lambda
outward. However, when the function parameters of an inner lambda depend on the
function parameters of an outer lambda, it can lead to
LYP951018 wrote:
@philnik777 It seems that clang crashes on the test
SemaCXX/cxx2a-template-lambdas.cpp https://godbolt.org/z/nnjzKr7n4
Is this expected?
https://github.com/llvm/llvm-project/pull/73376
___
cfe-commits mailing list
cfe-commits@lists.ll
27 matches
Mail list logo
