AmrDeveloper wrote:
> #121240
Sure, i am interested, Thank you :D
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
By the way, in case you're interested in doing another one, I filed
https://github.com/llvm/llvm-project/issues/121240 about another missing
matcher I came across.
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits
HighCommander4 wrote:
Went ahead and merged this. Thanks @AmrDeveloper!
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 approved this pull request.
Thanks, LGTM!
I will wait a few days before merging in case anyone else has feedback.
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
@@ -556,6 +556,22 @@ TEST_P(ASTMatchersTest, DeclRefExpr) {
Reference));
}
+TEST_P(ASTMatchersTest, DependentScopeDeclRefExpr) {
+ if (!GetParam().isCXX() || GetParam().hasDelayedTemplateParsing()) {
+// FIXME: Add a test for `dependentScopeDeclRe
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/120996
>From a0b71504fa31e55d42cb54e349838b2669f6fde5 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Mon, 23 Dec 2024 19:42:30 +0100
Subject: [PATCH 1/2] [Clang][ASTMatcher] Add `dependentScopeDeclRefExpr`
Mat
@@ -556,6 +556,22 @@ TEST_P(ASTMatchersTest, DeclRefExpr) {
Reference));
}
+TEST_P(ASTMatchersTest, DependentScopeDeclRefExpr) {
+ if (!GetParam().isCXX() || GetParam().hasDelayedTemplateParsing()) {
+// FIXME: Add a test for `dependentScopeDeclRe
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 commented:
Thanks, this looks pretty good!
https://github.com/llvm/llvm-project/pull/120996
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/120996
>From a0b71504fa31e55d42cb54e349838b2669f6fde5 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Mon, 23 Dec 2024 19:42:30 +0100
Subject: [PATCH] [Clang][ASTMatcher] Add `dependentScopeDeclRefExpr` Matcher
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/120996
>From 56b650ffe1634833b3147261a32b7b2d1c84c079 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Mon, 23 Dec 2024 19:42:30 +0100
Subject: [PATCH] [Clang][ASTMatcher] Add `dependentScopeDeclRefExpr` Matcher
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 11676da80874787a47856da87911234837c53f06
0e6e3334dafa4aa3a613ff996d55fb452b3fc350 --e
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Amr Hesham (AmrDeveloper)
Changes
Add AST Matcher for `DependentScopeDeclRefExpr`
Fixes: #120937
---
Full diff: https://github.com/llvm/llvm-project/pull/120996.diff
7 Files Affected:
- (modified) clang/docs/LibASTMatchersReference.htm
https://github.com/AmrDeveloper created
https://github.com/llvm/llvm-project/pull/120996
Add AST Matcher for `DependentScopeDeclRefExpr`
Fixes: #120937
>From 0e6e3334dafa4aa3a613ff996d55fb452b3fc350 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Mon, 23 Dec 2024 19:42:30 +0100
Subject: [PA
15 matches
Mail list logo
