https://github.com/tJener closed https://github.com/llvm/llvm-project/pull/87673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -510,6 +519,26 @@ int c = M3(3);
Visitor.runOver(Code.code());
}
+TEST(SourceCodeTest, InnerNestedTemplate) {
tJener wrote:
Done.
https://github.com/llvm/llvm-project/pull/87673
___
cfe-commits mailing list
https://github.com/tJener updated
https://github.com/llvm/llvm-project/pull/87673
>From 697551dd155cd8ada0d93ce8aa2951a9f9f3d4b4 Mon Sep 17 00:00:00 2001
From: Eric Li
Date: Thu, 4 Apr 2024 14:04:25 -0400
Subject: [PATCH 1/3] [libTooling] Fix `getFileRangeForEdit` for inner nested
template
@@ -510,6 +519,26 @@ int c = M3(3);
Visitor.runOver(Code.code());
}
+TEST(SourceCodeTest, InnerNestedTemplate) {
ymand wrote:
Add a test case that covers the case of the split being in the BeginToken?
https://github.com/llvm/llvm-project/pull/87673
https://github.com/ymand approved this pull request.
Nice work - really subtle! I'd been wondering how clang handles this issue (of
relexing returning the wrong token because it lacked context).
https://github.com/llvm/llvm-project/pull/87673
___
https://github.com/ymand edited https://github.com/llvm/llvm-project/pull/87673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tJener edited https://github.com/llvm/llvm-project/pull/87673
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tJener updated
https://github.com/llvm/llvm-project/pull/87673
>From 697551dd155cd8ada0d93ce8aa2951a9f9f3d4b4 Mon Sep 17 00:00:00 2001
From: Eric Li
Date: Thu, 4 Apr 2024 14:04:25 -0400
Subject: [PATCH 1/2] [libTooling] Fix `getFileRangeForEdit` for inner nested
template