Author: Chris B
Date: 2025-03-09T16:01:12-05:00
New Revision: e85e29c2992b3b86f9ac7db18c0c2b9dde95a2b2
URL:
https://github.com/llvm/llvm-project/commit/e85e29c2992b3b86f9ac7db18c0c2b9dde95a2b2
DIFF:
https://github.com/llvm/llvm-project/commit/e85e29c2992b3b86f9ac7db18c0c2b9dde95a2b2.diff
LOG:
https://github.com/vbvictor updated
https://github.com/llvm/llvm-project/pull/126434
>From 45d80294dd6fdab4bd85d51b0f952b515486bc8b Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Mon, 3 Mar 2025 09:25:03 +0300
Subject: [PATCH 1/3] [clang-tidy] add scoped-lock-check
---
.../clang-tidy/mod
YutongZhuu wrote:
> Can you update the first comment on this PR to reflect what you want to be in
> the commit message? I notice that it doesn't describe what you're doing with
> negation yet. Thanks!
Hello, can you merge this or delete the review request I sent? This was due to
an unsuccessf
https://github.com/a-tarasyuk closed
https://github.com/llvm/llvm-project/pull/123667
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AZero13 created
https://github.com/llvm/llvm-project/pull/130528
None
>From 340c458523af7ab6a3228f2173a4e95b3f7b9d67 Mon Sep 17 00:00:00 2001
From: Rose
Date: Sun, 9 Mar 2025 20:05:38 -0400
Subject: [PATCH] Handle indexes
---
clang/lib/AST/DeclCXX.cpp | 23
jvoung wrote:
It would be nice to to avoid having each project configure via
`AttributeMacros`.
The Abseil macro is meant to be a portable way to use the Clang-specific
nullability attributes like `_Nonnull` (portable in that it will be empty for
non-Clang compilers). So one would expect it t
efriedma-quic wrote:
/cherry-pick 42d49a77241df73a17cb442973702fc460e7fb90
https://github.com/llvm/llvm-project/pull/129952
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vinay-deshmukh updated
https://github.com/llvm/llvm-project/pull/129979
>From e9c7869550d9fd1eba4d4d42ee644540e6b6d445 Mon Sep 17 00:00:00 2001
From: Vinay Deshmukh <32487576+vinay-deshm...@users.noreply.github.com>
Date: Tue, 4 Mar 2025 22:30:34 -0500
Subject: [PATCH 1/8] add
@@ -151,6 +151,15 @@ struct Config {
ArgumentListsPolicy ArgumentLists = ArgumentListsPolicy::FullPlaceholders;
} Completion;
+ enum class HeaderInsertionPolicy {
+IWYU, // Include what you use
+NeverInsert // Never insert headers as part of code completio
https://github.com/HighCommander4 requested changes to this pull request.
Thanks for the patch! The general approach here looks good.
One issue that I see with the current implementation is that it makes the
command-line flag `--header-insertion=never` have no effect. That's because,
while the
https://github.com/HighCommander4 edited
https://github.com/llvm/llvm-project/pull/128503
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Aiden Grossman (boomanaiden154)
Changes
This patch makes the assertion (that is currently in a comment) that validates
that names mangled by clang can be demangled by LLVM actually compile/work.
There were some minor issues that n
https://github.com/efriedma-quic closed
https://github.com/llvm/llvm-project/pull/129952
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hstk30-hw closed
https://github.com/llvm/llvm-project/pull/130129
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ryan Mansfield
Date: 2025-03-10T09:37:21+08:00
New Revision: d111e6481afa71fedc0a1877d20943e43954edce
URL:
https://github.com/llvm/llvm-project/commit/d111e6481afa71fedc0a1877d20943e43954edce
DIFF:
https://github.com/llvm/llvm-project/commit/d111e6481afa71fedc0a1877d20943e43954edce.diff
https://github.com/Endilll commented:
Changes to C++ DR tests look good to me
https://github.com/llvm/llvm-project/pull/130529
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,319 @@
+//===--- UseScopedLockCheck.cpp - clang-tidy
--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/126434
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vinay-deshmukh updated
https://github.com/llvm/llvm-project/pull/129979
>From e9c7869550d9fd1eba4d4d42ee644540e6b6d445 Mon Sep 17 00:00:00 2001
From: Vinay Deshmukh <32487576+vinay-deshm...@users.noreply.github.com>
Date: Tue, 4 Mar 2025 22:30:34 -0500
Subject: [PATCH 1/6] add
https://github.com/YutongZhuu edited
https://github.com/llvm/llvm-project/pull/126846
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ryosuke Niwa
Date: 2025-03-09T14:59:46-07:00
New Revision: 5c3b05996fa5a076bc086f2eeca56f080f255754
URL:
https://github.com/llvm/llvm-project/commit/5c3b05996fa5a076bc086f2eeca56f080f255754
DIFF:
https://github.com/llvm/llvm-project/commit/5c3b05996fa5a076bc086f2eeca56f080f255754.diff
https://github.com/rniwa closed https://github.com/llvm/llvm-project/pull/128651
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/126434
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9180,6 +9180,12 @@ bool Sema::hasAcceptableDefinition(NamedDecl *D,
NamedDecl **Suggested,
if (!getLangOpts().Modules && !getLangOpts().ModulesLocalVisibility)
return true;
+ // The external source may have additional definitions of this entity that
are
+ // visi
llvmbot wrote:
Failed to cherry-pick: 42d49a77241df73a17cb442973702fc460e7fb90
https://github.com/llvm/llvm-project/actions/runs/13755084406
Please manually backport the fix and push it to your github fork. Once this is
done, please create a [pull
request](https://github.com/llvm/llvm-proje
https://github.com/t-rasmud approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/128641
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ojhunt wrote:
@cor3ntin @AaronBallman @erichkeane There is a bunch of super unpleasant
nonsense in the new and delete codegen paths, it would simplify things a lot if
we actually had the AST nodes actually be complete, e.g. the AST node would
include expression nodes for the implicit parameter
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-x86_64-linux-android` running on `sanitizer-buildbot-android` while
building `clang,llvm` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/186/builds/7174
Here is the r
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/130473
>From 66e5ed1f1fcfe5a52d110ae228ddd51bd487ee16 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
@@ -9180,6 +9180,12 @@ bool Sema::hasAcceptableDefinition(NamedDecl *D,
NamedDecl **Suggested,
if (!getLangOpts().Modules && !getLangOpts().ModulesLocalVisibility)
return true;
+ // The external source may have additional definitions of this entity that
are
+ // visi
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/128586
>From 90403f77295c0b3e4415263754e22b3c66f31876 Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Mon, 24 Feb 2025 13:43:04 -0800
Subject: [PATCH 1/6] [alpha.webkit.UnretainedCallArgsChecker] Add a checker
for NS
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/130228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Ryosuke Niwa (rniwa)
Changes
Add a new static analyzer which emits warnings for function call arguments,
local variables, and member variables that are only forward declared. These
forward declaration prevents other WebK
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ryosuke Niwa (rniwa)
Changes
Add a new static analyzer which emits warnings for function call arguments,
local variables, and member variables that are only forward declared. These
forward declaration prevents other WebKit checkers from c
https://github.com/rniwa created
https://github.com/llvm/llvm-project/pull/130554
Add a new static analyzer which emits warnings for function call arguments,
local variables, and member variables that are only forward declared. These
forward declaration prevents other WebKit checkers from chec
@@ -9180,6 +9180,12 @@ bool Sema::hasAcceptableDefinition(NamedDecl *D,
NamedDecl **Suggested,
if (!getLangOpts().Modules && !getLangOpts().ModulesLocalVisibility)
return true;
+ // The external source may have additional definitions of this entity that
are
+ // visi
rniwa wrote:
Thanks for the review!
https://github.com/llvm/llvm-project/pull/128641
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HighCommander4 wrote:
I split the PR up into a few commits for easier reviewing.
The high-level idea here is that the helper function `getAsRecordDecl()` in
SemaCodeComplete.cpp is doing similar things to HeuristicResolver
(specifically, `simplifyType` and `resolveTypeToRecordDecl`), but
Heur
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nathan Ridge (HighCommander4)
Changes
Fixes https://github.com/llvm/llvm-project/issues/130468
---
Full diff: https://github.com/llvm/llvm-project/pull/130473.diff
6 Files Affected:
- (modified) clang/include/clang/Sema/HeuristicResolve
https://github.com/boomanaiden154 closed
https://github.com/llvm/llvm-project/pull/130522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dl8sd11 created
https://github.com/llvm/llvm-project/pull/130540
Support float and long double versions of the math functions for
UseStdNumbersCheck.
For example, after this commit the check is able to catch `sqrtf(2)` and
`expl(1)`.
Fixes: #130325
>From 092135bbb3536167f
Author: Aiden Grossman
Date: 2025-03-09T19:56:40-07:00
New Revision: de132b2a0c344729703358a1c3ac90fb28fdb130
URL:
https://github.com/llvm/llvm-project/commit/de132b2a0c344729703358a1c3ac90fb28fdb130
DIFF:
https://github.com/llvm/llvm-project/commit/de132b2a0c344729703358a1c3ac90fb28fdb130.diff
https://github.com/dl8sd11 updated
https://github.com/llvm/llvm-project/pull/130540
>From 092135bbb3536167f0cad11e7320e52886c022cc Mon Sep 17 00:00:00 2001
From: dl8sd11
Date: Mon, 10 Mar 2025 02:56:14 +
Subject: [PATCH] [clang-tidy] support different precisions
Support float and long doub
https://github.com/vinay-deshmukh updated
https://github.com/llvm/llvm-project/pull/129979
>From e9c7869550d9fd1eba4d4d42ee644540e6b6d445 Mon Sep 17 00:00:00 2001
From: Vinay Deshmukh <32487576+vinay-deshm...@users.noreply.github.com>
Date: Tue, 4 Mar 2025 22:30:34 -0500
Subject: [PATCH 01/11] a
https://github.com/vinay-deshmukh edited
https://github.com/llvm/llvm-project/pull/129979
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Tommy Chen (dl8sd11)
Changes
Support float and long double versions of the math functions for
UseStdNumbersCheck.
For example, after this commit the check is able to catch `sqrtf(2)` and
`expl(1)`.
Fixes: #130325
---
Full diff: ht
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/130554
>From d2edd9b5fd188476e59790c333497516933532bb Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Sun, 9 Mar 2025 23:06:33 -0700
Subject: [PATCH 1/2] [alpha.webkit.ForwardDeclChecker] Add a new WebKit
checker for
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/130554
>From d2edd9b5fd188476e59790c333497516933532bb Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Sun, 9 Mar 2025 23:06:33 -0700
Subject: [PATCH] [alpha.webkit.ForwardDeclChecker] Add a new WebKit checker
for forw
@@ -892,6 +892,11 @@ namespace lambdas {
return a1.x;
});
A a2([&] { return a2.x; }); // ok
+A a3([=] { return a3.x; }()); // expected-warning{{variable 'a3' is
uninitialized when used within its own initialization}}
zhaohuiw42 wrote:
I thin
@@ -12609,6 +12611,7 @@ namespace {
isPODType = false;
isRecordType = false;
isReferenceType = false;
+ isInCXXOperatorCall = false;
zhaohuiw42 wrote:
So I need to change all the other fields to this form as well, like
`isPODType`, `isR
Author: Oleksandr T.
Date: 2025-03-10T01:53:20+02:00
New Revision: bfdeb5873073ed94b63b1ef1f7e91845c1be88a1
URL:
https://github.com/llvm/llvm-project/commit/bfdeb5873073ed94b63b1ef1f7e91845c1be88a1
DIFF:
https://github.com/llvm/llvm-project/commit/bfdeb5873073ed94b63b1ef1f7e91845c1be88a1.diff
https://github.com/mizvekov updated
https://github.com/llvm/llvm-project/pull/130447
>From 78ac2f4cbc45c4f77aab5e28e3450e9c459b8077 Mon Sep 17 00:00:00 2001
From: Matheus Izvekov
Date: Sat, 8 Mar 2025 20:32:14 -0300
Subject: [PATCH] [clang] fix matching of nested template template parameters
W
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: AZero13 (AZero13)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/130528.diff
1 Files Affected:
- (modified) clang/lib/AST/DeclCXX.cpp (+22-1)
``diff
diff --git a/clang/lib/AST/DeclCXX.cpp b/clang/lib/AST/De
https://github.com/mizvekov created
https://github.com/llvm/llvm-project/pull/130529
This clears up the printing of a NestedNameSpecifier so a trailing '::' is not
printed, unless it refers into the global scope.
This fixes a bunch of diagnostics where the trailing :: was awkward. This also
p
https://github.com/AZero13 closed
https://github.com/llvm/llvm-project/pull/130528
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/130192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/128641
>From a4cd301d6176a8ee3441d0c74f6d2c4e32a50cdd Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Mon, 24 Feb 2025 23:01:31 -0800
Subject: [PATCH 1/3] [alpha.webkit.NoUnretainedMemberChecker] Add a new WebKit
chec
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/130228
>From 295b8173b6913d9014c5786eb4af0112384afa65 Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Fri, 7 Mar 2025 11:38:11 +0800
Subject: [PATCH 1/4] [Clang] Implement P0963R3 "Structured binding declaration
as
DeNiCoN wrote:
> Hello @DeNiCoN, Are you still working on this patch? If not, I hope I can
> take over this patch to finally finish it.
Hello @HerrCai0907. I'm not actively working on it. You can take over
Some notes:
I've found a bug that if a config file is not explicitly specified(no
`--co
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Congcong Cai (HerrCai0907)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/130493.diff
1 Files Affected:
- (modified) clang-tools-extra/clang-tidy/misc/ConstCorrectnessCheck.cpp (+6-8)
``diff
diff --git
https://github.com/boomanaiden154 created
https://github.com/llvm/llvm-project/pull/130522
This patch makes the assertion (that is currently in a comment) that validates
that names mangled by clang can be demangled by LLVM actually compile/work.
There were some minor issues that needed to be f
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Matheus Izvekov (mizvekov)
Changes
This clears up the printing of a NestedNameSpecifier so a trailing '::' is not
printed, unless it refers into the global scope.
This fixes a bunch of diagnostics where the trailing :: was awkward. This a
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-armv8-quick` running
on `linaro-clang-armv8-quick` while building `clang` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/154/builds/13059
Here is the relevant piece o
MythreyaK wrote:
> One issue that I see with the current implementation is that it makes the
> command-line flag `--header-insertion=never` have no effect.
Oh right! I completely forgot about that.
> To get the command line flag to interact with the config option better, we
> need to handle
HighCommander4 wrote:
> > To get the command line flag to interact with the config option better, we
> > need to handle it in
> > [`FlagsConfigProvider`](https://searchfox.org/llvm/rev/107aa6a3d3ab96b7eec55e1ec5c3eabfa6ab2f9f/clang-tools-extra/clangd/tool/ClangdMain.cpp#655),
> > which gives t
https://github.com/efriedma-quic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/130522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vinay-deshmukh updated
https://github.com/llvm/llvm-project/pull/129979
>From e9c7869550d9fd1eba4d4d42ee644540e6b6d445 Mon Sep 17 00:00:00 2001
From: Vinay Deshmukh <32487576+vinay-deshm...@users.noreply.github.com>
Date: Tue, 4 Mar 2025 22:30:34 -0500
Subject: [PATCH 1/7] add
https://github.com/vinay-deshmukh updated
https://github.com/llvm/llvm-project/pull/129979
>From e9c7869550d9fd1eba4d4d42ee644540e6b6d445 Mon Sep 17 00:00:00 2001
From: Vinay Deshmukh <32487576+vinay-deshm...@users.noreply.github.com>
Date: Tue, 4 Mar 2025 22:30:34 -0500
Subject: [PATCH 1/8] add
Author: Eli Friedman
Date: 2025-03-09T18:38:55-07:00
New Revision: 42d49a77241df73a17cb442973702fc460e7fb90
URL:
https://github.com/llvm/llvm-project/commit/42d49a77241df73a17cb442973702fc460e7fb90
DIFF:
https://github.com/llvm/llvm-project/commit/42d49a77241df73a17cb442973702fc460e7fb90.diff
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Tommy Chen (dl8sd11)
Changes
Support float and long double versions of the math functions for
UseStdNumbersCheck.
For example, after this commit the check is able to catch `sqrtf(2)` and
`expl(1)`.
Fixes: #130325
---
Full d
https://github.com/dl8sd11 ready_for_review
https://github.com/llvm/llvm-project/pull/130540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vinay-deshmukh updated
https://github.com/llvm/llvm-project/pull/129979
>From e9c7869550d9fd1eba4d4d42ee644540e6b6d445 Mon Sep 17 00:00:00 2001
From: Vinay Deshmukh <32487576+vinay-deshm...@users.noreply.github.com>
Date: Tue, 4 Mar 2025 22:30:34 -0500
Subject: [PATCH 01/10] a
https://github.com/YutongZhuu edited
https://github.com/llvm/llvm-project/pull/126846
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 85d60a441ab810e25605fb4555971b1d0a996e5c
340c458523af7ab6a3228f2173a4e95b3f7b9d67 --e
Author: Zahira Ammarguellat
Date: 2025-03-06T07:51:45-05:00
New Revision: 7358973df1b53785cace2e3431d141078dc53e7c
URL:
https://github.com/llvm/llvm-project/commit/7358973df1b53785cace2e3431d141078dc53e7c
DIFF:
https://github.com/llvm/llvm-project/commit/7358973df1b53785cace2e3431d141078dc53e7c
101 - 175 of 175 matches
Mail list logo
