HighCommander4 wrote:
> Any update about the issue that I reported above?
You may want to open a new issue to track that, so it can get annotated with
the appropriate labels (e.g. `rejects-valid`) and get more attention by showing
up on issues lists / dashboards that people may be watching.
h
https://github.com/HighCommander4 approved this pull request.
Thanks!
(Previously reviewed in https://github.com/llvm/llvm-project/pull/80180.)
https://github.com/llvm/llvm-project/pull/141208
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
@@ -262,6 +264,8 @@ TEST(IncludeCleaner, GenerateMissingHeaderDiags) {
TU.Filename = "main.cpp";
TU.AdditionalFiles["a.h"] = guard("#include \"b.h\"");
TU.AdditionalFiles["b.h"] = guard("void b();");
+ TU.AdditionalFiles["a_angled.h"] = guard("#include \"b_angled.h\"");
https://github.com/HighCommander4 requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/140594
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> > But since we have both config options, maybe there is a reason for
> > overwrites in this direction as well.
>
> Hmm, I was mainly focusing on the quoted -> angled case, hence why only the
> `AngledHeaders` filter is used. But I have to agree that following the config
@@ -297,7 +301,8 @@ TEST(IncludeCleaner, GenerateMissingHeaderDiags) {
Findings.UnusedIncludes.clear();
std::vector Diags = issueIncludeCleanerDiagnostics(
AST, TU.Code, Findings, MockFS(),
- {[](llvm::StringRef Header) { return Header.ends_with("buzz.h"); }});
+
@@ -306,6 +311,12 @@ TEST(IncludeCleaner, GenerateMissingHeaderDiags) {
withFix({Fix(MainFile.range("insert_b"), "#include \"b.h\"\n",
"#include \"b.h\""),
FixMessage("add all missing includes")})),
+
@@ -262,6 +264,8 @@ TEST(IncludeCleaner, GenerateMissingHeaderDiags) {
TU.Filename = "main.cpp";
TU.AdditionalFiles["a.h"] = guard("#include \"b.h\"");
TU.AdditionalFiles["b.h"] = guard("void b();");
+ TU.AdditionalFiles["a_angled.h"] = guard("#include \"b_angled.h\"");
@@ -142,6 +143,13 @@ std::vector generateMissingIncludeDiagnostics(
llvm::StringRef HeaderRef{Spelling};
bool Angled = HeaderRef.starts_with("<");
+for (auto Filter : AngledHeaders) {
HighCommander4 wrote:
nit: iterate using `auto &`, the way we d
https://github.com/HighCommander4 converted_to_draft
https://github.com/llvm/llvm-project/pull/140651
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> I am afraid this doesn't address
> [clangd/clangd#1104](https://github.com/clangd/clangd/issues/1104) to a
> useful extent :/.
>
> Whenever a header `foo.h` changes, clangd will schedule indexing of only a
> **single** translation unit that depends on it, not all of th
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/140651
>From 7f48d903c6d5f97a75208c799e0e96a26948af05 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sat, 17 May 2025 22:11:45 -0400
Subject: [PATCH 1/2] [clangd] Drop the optimization where only shards for
HighCommander4 wrote:
> @HighCommander4 I'm just wondering, have you had a chance to check out this
> PR yet? If not, is there any chance you could find some time for it in the
> coming weeks or months?
My apologies, this has been languishing in my review queue and I haven't quite
managed to
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/140651
>From 7f48d903c6d5f97a75208c799e0e96a26948af05 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sat, 17 May 2025 22:11:45 -0400
Subject: [PATCH] [clangd] Drop the optimization where only shards for modif
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/140651
The optimization is not valid, given that the target of a symbol reference can
change without the spelling of the reference (or anything else in the file
containing the reference) changing, as illustrate
HighCommander4 wrote:
> @kleinesfilmroellchen @HighCommander4 would it be possible to take a look
> when you have some time? Many thanks!
Yep, will do. Added myself as reviewer.
https://github.com/llvm/llvm-project/pull/140594
___
cfe-commits mailing
HighCommander4 wrote:
> If you're looking for a clangd reviewer, perhaps one of our other clangd
> maintainers, @llvm-beanz or @kadircet, could help with this one.
(I realized you are probably not able to edit the Reviewers field, so I went
ahead and added the mentioned folks.)
https://github
HighCommander4 wrote:
> LGTM if HighCommander4 is happy.
Apologies, folks, I'm backlogged on reviews and probably won't have time to
look at this for a while. If you're looking for a clangd reviewer, perhaps one
of our other clangd maintainers, @llvm-beanz or @kadircet, could help with this
o
HighCommander4 wrote:
@Noustaa if you'd like to also submit a documentation patch for
https://clangd.llvm.org/config.html (source at
https://github.com/llvm/clangd-www/blob/main/config.md), that would be
appreciated!
https://github.com/llvm/llvm-project/pull/137613
___
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/137613
___
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 made one small tweak, to ConfigFragment.h, to refer to the config option
values `All` and `None` with correct capitalization.)
https://github.com/llvm/llvm-project/pull/137613
___
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/137613
>From 7e715447d242d7e409e02059d2948fb3417fb00f Mon Sep 17 00:00:00 2001
From: Noustaa
Date: Mon, 28 Apr 2025 12:38:36 +0200
Subject: [PATCH 1/5] [clangd] Add CodePatterns config option under Completion
A
@@ -267,6 +267,17 @@ opt HeaderInsertion{
"Never insert #include directives as part of code completion")),
};
+opt CodePatterns{
HighCommander4 wrote:
We've been moving towards having configuration specified only in the config
file, not on the co
@@ -950,6 +951,11 @@ struct CompletionRecorder : public CodeCompleteConsumer {
// Retain the results we might want.
for (unsigned I = 0; I < NumResults; ++I) {
auto &Result = InResults[I];
+ if (Config::current().Completion.CodePatterns ==
+ Conf
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/137613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 requested changes to this pull request.
Thanks, the patch and tests look pretty good. I think we should leave out the
command-line arg, but otherwise should be good to go.
https://github.com/llvm/llvm-project/pull/137613
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/139489
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 requested changes to this pull request.
Looks reasonable to me, but the buildkite run is showing
`AllowImplicitCompletions.All` is currently failing
https://github.com/llvm/llvm-project/pull/138846
___
cfe-commits mai
@@ -2455,6 +2455,11 @@ bool isIncludeFile(llvm::StringRef Line) {
}
bool allowImplicitCompletion(llvm::StringRef Content, unsigned Offset) {
+ // Check if we're inside an empty template argument list
HighCommander4 wrote:
nit: for consistency with the other
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/138846
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
(This came up during https://github.com/llvm/llvm-project/issues/139019 as a
potential diagnosis, but it does not fix the crash. As mentioned, no one was in
fact moving from the arguments, so passing them by rvalue reference and
forwarding them multiple times in the same
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/139489
The arguments passed are lightweight (an ArrayRef and a pointer), and
findSpecializationImpl passes them to multiple functions, making it a potential
hazard to pass them by rvalue reference (even though
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/138640
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/138640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/138640
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial
@@ -25,6 +26,17 @@ static bool isGeneratedDecl(const Decl *D) {
return false;
}
+IndexingContext::IndexingContext(IndexingOptions IndexOpts,
+ IndexDataConsumer &DataConsumer)
+: IndexOpts(IndexOpts), DataConsumer(DataConsumer) {}
+
+Index
HighCommander4 wrote:
Per discussion at
https://github.com/llvm/llvm-project/pull/125153#discussion_r2074478720
https://github.com/llvm/llvm-project/pull/138640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/138640
None
>From eeeb71e72e3d5e9da5227b0f2f1d6e870e393a7c Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Tue, 6 May 2025 02:50:47 -0400
Subject: [PATCH] [clang][Index] Add comment about out-of-line defaulte
@@ -25,6 +26,17 @@ static bool isGeneratedDecl(const Decl *D) {
return false;
}
+IndexingContext::IndexingContext(IndexingOptions IndexOpts,
+ IndexDataConsumer &DataConsumer)
+: IndexOpts(IndexOpts), DataConsumer(DataConsumer) {}
+
+Index
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/133681
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> There is 2 different features involved here.
>
> [...]
>
> In the feature i'm trying to implement, number 1 should be OFF if
> CodePatterns is set to None but number 2 should still be working.
That makes sense to me, as a user expectation.
The code I linked to in my pr
@@ -158,6 +159,42 @@ TEST(StdLibTests, EndToEnd) {
UnorderedElementsAre(StdlibSymbol("list"), StdlibSymbol("vector")));
}
+TEST(StdLibTests, StdLibDocComments) {
+ Config Cfg;
+ Cfg.Index.StandardLibrary = true;
+ WithContextValue Enabled(Config::Key, std::move(Cfg));
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/133681
>From ce15317543b135b7aa86d4987fb0c25abd597d56 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Mon, 31 Mar 2025 02:25:45 -0400
Subject: [PATCH] [clangd] Store documentation when indexing standard librar
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/71605
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
Thanks! I'll go ahead and merge.
https://github.com/llvm/llvm-project/pull/71605
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> Code autocomplete works just fine with CodePatterns::None but include
> autocomplete does not work anymore. Gonna investigate when i have some time.
I guess the preprocessor completions are all `RK_Pattern` (added
[here](https://searchfox.org/llvm/rev/2f976956e5ccef5664
@@ -158,6 +158,43 @@ TEST(StdLibTests, EndToEnd) {
UnorderedElementsAre(StdlibSymbol("list"), StdlibSymbol("vector")));
}
+TEST(StdLibTests, StdLibDocComments) {
+ Config Cfg;
+ Cfg.Index.StandardLibrary = true;
+ WithContextValue Enabled(Config::Key, std::move(Cfg));
@@ -223,33 +226,26 @@ SymbolSlab indexStandardLibrary(llvm::StringRef
HeaderSources,
return Symbols;
}
- SymbolCollector::Options IndexOpts;
- IndexOpts.Origin = SymbolOrigin::StdLib;
- IndexOpts.CollectMainFileSymbols = false;
- IndexOpts.CollectMainFileRefs = fals
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/133681
>From 451a88947bda87daa5934cc5c2d55f9e57a5cfda Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Mon, 31 Mar 2025 02:25:45 -0400
Subject: [PATCH] [clangd] Store documentation when indexing standard librar
HighCommander4 wrote:
Addressed review comments
https://github.com/llvm/llvm-project/pull/133681
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/133681
>From aab6aef62b7691290f02b0dd29b0071f428502c9 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Mon, 31 Mar 2025 02:25:45 -0400
Subject: [PATCH] [clangd] Store documentation when indexing standard librar
HighCommander4 wrote:
> Yes this is good to go
Could you rebase it to apply to latest trunk please? I'm happy to go ahead and
merge it then.
https://github.com/llvm/llvm-project/pull/71605
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
HighCommander4 wrote:
Thanks; I'll go ahead and merge.
https://github.com/llvm/llvm-project/pull/134657
___
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/134657
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> unfortunately, LLVM's command-line argument framework doesn't allow us to
> distinguish between "the option's value is `IWYU` because the flag
> `--header-insertion=iwyu` appeared in the command line" and "the option's
> value is `IWYU` because no `--header-insertion` f
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/136106
___
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.
Looks good, thank you!
https://github.com/llvm/llvm-project/pull/136106
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/136106
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2305,7 +2335,45 @@ TEST(BlockEndHints, PointerToMemberFunction) {
$ptrmem[[}]]
} // suppress
)cpp",
- ExpectedHint{" // if", "ptrmem"});
+ ExpectedHint{" // if ()", "ptrmem"});
+}
+
+TEST(BlockEndHints, MinLineLimit) {
+
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/136106
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/136106
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/136106
___
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/136237
___
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!
https://github.com/llvm/llvm-project/pull/136237
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -22,10 +22,16 @@ namespace clang {
namespace clangd {
class ParsedAST;
+struct InlayHintOptions {
+ // Minimum lines for BlockEnd inlay-hints to be shown
HighCommander4 wrote:
Let's make this comment a bit more specific:
```
// Minimum height of a code b
@@ -2305,7 +2335,45 @@ TEST(BlockEndHints, PointerToMemberFunction) {
$ptrmem[[}]]
} // suppress
)cpp",
- ExpectedHint{" // if", "ptrmem"});
+ ExpectedHint{" // if ()", "ptrmem"});
+}
+
+TEST(BlockEndHints, MinLineLimit) {
+
@@ -22,10 +22,16 @@ namespace clang {
namespace clangd {
class ParsedAST;
+struct InlayHintOptions {
+ // Minimum lines for BlockEnd inlay-hints to be shown
+ int HintMinLineLimit{2};
HighCommander4 wrote:
What we discussed in the original issue was that th
@@ -2305,7 +2335,45 @@ TEST(BlockEndHints, PointerToMemberFunction) {
$ptrmem[[}]]
} // suppress
)cpp",
- ExpectedHint{" // if", "ptrmem"});
+ ExpectedHint{" // if ()", "ptrmem"});
+}
+
+TEST(BlockEndHints, MinLineLimit) {
+
https://github.com/HighCommander4 requested changes to this pull request.
Thanks for picking up this patch!
Looks fairly good, just a few comments:
https://github.com/llvm/llvm-project/pull/136106
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
@@ -36,9 +36,12 @@ namespace {
using ::testing::ElementsAre;
using ::testing::IsEmpty;
-std::vector hintsOfKind(ParsedAST &AST, InlayHintKind Kind) {
+constexpr InlayHintOptions DefaultInlayHintOpts{};
HighCommander4 wrote:
nit: let's call this `DefaultInlayH
@@ -147,6 +149,9 @@ std::string summarizeExpr(const Expr *E) {
}
// Literals are just printed
+std::string VisitCXXNullPtrLiteralExpr(const CXXNullPtrLiteralExpr *E) {
+ return "nullptr";
HighCommander4 wrote:
Could you add a small test case
https://github.com/HighCommander4 approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/135620
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
> I was talking about a change like
> [kadircet@ff0c31d](https://github.com/kadircet/llvm-project/commit/ff0c31d232b2aed9e95d69d16a9dfbb9babea711)
Thanks for the more fleshed-out suggestion.
As written, this would also have the following effects:
1. we no longer use `Sy
HighCommander4 wrote:
@tdupes @kadircet it looks like this PR was approved but has not been merged --
could we merge it?
https://github.com/llvm/llvm-project/pull/71605
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/133681
>From bd514d4ab9a593f33ef36f9bc3fa888a17b633d7 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Mon, 31 Mar 2025 02:25:45 -0400
Subject: [PATCH] [clangd] Store documentation when indexing standard librar
HighCommander4 wrote:
The described options seem a bit more involved than necessary to fix this bug,
given that it's just the value of the `StoreAllDocumentation` flag that's a
problem.
I revised the patch to add a new parameter to `createStaticIndexingAction()`
(now called `createIndexingAct
HighCommander4 wrote:
> Yes, would be great to see it get backported
Filed backport issue: https://github.com/llvm/llvm-project/issues/135922
https://github.com/llvm/llvm-project/pull/135757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/135757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/135757
>From a1569727c432ba67a96de07c899b562fcf349d1e Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Tue, 15 Apr 2025 03:40:37 -0400
Subject: [PATCH] [clang][AST] Handle implicit first argument in
CallExpr::
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/135757
>From a1569727c432ba67a96de07c899b562fcf349d1e Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Tue, 15 Apr 2025 03:40:37 -0400
Subject: [PATCH] [clang][AST] Handle implicit first argument in
CallExpr::
HighCommander4 wrote:
Added changelog entry.
Should we backport this to 20.x?
https://github.com/llvm/llvm-project/pull/135757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1652,8 +1652,11 @@ SourceLocation CallExpr::getBeginLoc() const {
if (!isTypeDependent()) {
if (const auto *Method =
dyn_cast_if_present(getCalleeDecl());
-Method && Method->isExplicitObjectMemberFunction())
- return getArg(0)->getBeginLoc();
HighCommander4 wrote:
Review ping
https://github.com/llvm/llvm-project/pull/133681
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
Ok, I went ahead and merged it.
https://github.com/llvm/llvm-project/pull/133124
___
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/133124
___
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.
LGTM, thanks
https://github.com/llvm/llvm-project/pull/134951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
(Rebased)
https://github.com/llvm/llvm-project/pull/132576
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/132576
>From ac3780d5f747c8c063119a028a4735c725198003 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sat, 22 Mar 2025 20:54:02 -0400
Subject: [PATCH] [clang][HeuristicResolver] Apply default argument heuristi
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/132576
___
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/130473
___
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 for the patch!
I made a small addition to the test to make it clearer why we have this
conditional check. Please have a look at let me know if that makes sense to you.
https://github.com/llvm/llvm-project/pull/134657
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/134657
>From 162b0535fc2ae745441404231e6b3bf3882b974e Mon Sep 17 00:00:00 2001
From: Hampus Adolfsson
Date: Mon, 7 Apr 2025 14:51:03 +0200
Subject: [PATCH 1/2] [clangd] Strip invalid fromRanges for outgoing call
HighCommander4 wrote:
(Rebased)
https://github.com/llvm/llvm-project/pull/130473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/130473
>From f6c57947b12d8c22e8e54757fc03ff1e407aa12e Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sun, 9 Mar 2025 01:36:25 -0500
Subject: [PATCH 1/6] Change the parameter type of resolveTypeToTagDecl() to
@@ -54,8 +54,9 @@ IdentifierInfo *Parser::getSEHExceptKeyword() {
}
Parser::Parser(Preprocessor &pp, Sema &actions, bool skipFunctionBodies)
-: PP(pp), PreferredType(pp.isCodeCompletionEnabled()), Actions(actions),
- Diags(PP.getDiagnostics()), StackHandler(Diags),
+
HighCommander4 wrote:
Review ping :)
https://github.com/llvm/llvm-project/pull/130473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
@tcottin, thank you for taking the time to pick up this patch! It's clearly a
popular feature request.
@kadircet, you were involved with earlier rounds of review on the old patch
(https://reviews.llvm.org/D143112) and design discussions in the issue
(https://github.com/c
HighCommander4 wrote:
See diagnosis
[here](https://github.com/clangd/clangd/issues/2344#issuecomment-2765249487):
> The **actual** reason that `std::vector::size` is missing documentation when
> the standard library AST is indexed is that, unlike during preamble indexing,
> `Opts.StoreAllDocu
HighCommander4 wrote:
(I messed things up a bit here by merging a PR that was stacked on top of this
into this one. Marking as Draft until I sort that out.)
https://github.com/llvm/llvm-project/pull/130473
___
cfe-commits mailing list
cfe-commits@list
HighCommander4 wrote:
This was previously submitted and approved at
https://github.com/llvm/llvm-project/pull/131074, I just mistakenly merged that
into a user branch.
https://github.com/llvm/llvm-project/pull/132465
___
cfe-commits mailing list
cfe-
1 - 100 of 1055 matches
Mail list logo
