https://github.com/vbvictor updated
https://github.com/llvm/llvm-project/pull/148357
>From 82bee931621536e7e59910f1833289510909ce23 Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Sat, 12 Jul 2025 14:24:31 +0300
Subject: [PATCH] [clang-tidy] Use lexical anon-ns matcher in
llvm-prefer-stati
Author: Baranov Victor
Date: 2025-07-12T15:57:25+03:00
New Revision: bd7a6bfdf4d71f62ce7b536d543592ce73646366
URL:
https://github.com/llvm/llvm-project/commit/bd7a6bfdf4d71f62ce7b536d543592ce73646366
DIFF:
https://github.com/llvm/llvm-project/commit/bd7a6bfdf4d71f62ce7b536d543592ce73646366.diff
https://github.com/vbvictor closed
https://github.com/llvm/llvm-project/pull/148352
___
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: None (Sirraide)
Changes
Revert llvm/llvm-project#143520 for now since it’s causing issues for
people who are using symlinks and prefer to preserve the original path (i.e.
looks like we’ll have to make this configurable after all; I j
Author: Yanzuo Liu
Date: 2025-07-12T21:13:30+08:00
New Revision: c4cc3573d144831d2815433646ffcab62cc9ea40
URL:
https://github.com/llvm/llvm-project/commit/c4cc3573d144831d2815433646ffcab62cc9ea40
DIFF:
https://github.com/llvm/llvm-project/commit/c4cc3573d144831d2815433646ffcab62cc9ea40.diff
LO
https://github.com/Sirraide created
https://github.com/llvm/llvm-project/pull/148367
Reverts llvm/llvm-project#143520 for now since it’s causing issues for people
who are using symlinks and prefer to preserve the original path (i.e. looks
like we’ll have to make this configurable after all; I
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/148195
___
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-tools-extra
Author: None (Sirraide)
Changes
Revert llvm/llvm-project#143520 for now since it’s causing issues for
people who are using symlinks and prefer to preserve the original path (i.e.
looks like we’ll have to make this configurable after a
Author: Yanzuo Liu
Date: 2025-07-12T21:13:30+08:00
New Revision: c4cc3573d144831d2815433646ffcab62cc9ea40
URL:
https://github.com/llvm/llvm-project/commit/c4cc3573d144831d2815433646ffcab62cc9ea40
DIFF:
https://github.com/llvm/llvm-project/commit/c4cc3573d144831d2815433646ffcab62cc9ea40.diff
LO
Author: Sirraide
Date: 2025-07-12T15:13:22+02:00
New Revision: 7b43c6c6a729bde8b20143f9620e6323812f1694
URL:
https://github.com/llvm/llvm-project/commit/7b43c6c6a729bde8b20143f9620e6323812f1694
DIFF:
https://github.com/llvm/llvm-project/commit/7b43c6c6a729bde8b20143f9620e6323812f1694.diff
LOG:
https://github.com/Sirraide closed
https://github.com/llvm/llvm-project/pull/148367
___
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
Author: None (Sirraide)
Changes
Revert llvm/llvm-project#143520 for now since it’s causing issues for
people who are using symlinks and prefer to preserve the original path (i.e.
looks like we’ll have to make this configurable after all; I just n
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/148367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
Thanks for your review.
https://github.com/llvm/llvm-project/pull/148195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-x86_64-debian`
running on `lldb-x86_64-debian` while building `clang-tools-extra,clang,lldb`
at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/162/builds/26685
Here is the releva
Sirraide wrote:
> do so by default, but disabled by "-no-canonical-prefix"
So just to be clear, while there might be a better place to canonicalise the
paths (we can’t treat include paths the same as the resource directory because
that’s canonicalised in `clang_main`), the main thing you care
jyknight wrote:
> the main thing you care about is being able to turn off the canonicalisation
> entirely with -no-canonical-prefix, is that right?
In terms of not breaking the Google buildsystem: yes. But if the question is
whether the current patch would be fine if its behavior was also disa
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/148025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Amr Hesham
Date: 2025-07-12T17:22:57+02:00
New Revision: 647f02a02a90038b7af1879cf96565f204a2c949
URL:
https://github.com/llvm/llvm-project/commit/647f02a02a90038b7af1879cf96565f204a2c949
DIFF:
https://github.com/llvm/llvm-project/commit/647f02a02a90038b7af1879cf96565f204a2c949.diff
LO
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Sirui Mu (Lancern)
Changes
This patch contains fixes for various nits mentioned in #147200:
- This patch removes the `bit.` prefix in the op mnemonic. The operation names
now directly correspond to the builtin function names except for `b
https://github.com/Lancern created
https://github.com/llvm/llvm-project/pull/148378
This patch contains fixes for various nits mentioned in #147200:
- This patch removes the `bit.` prefix in the op mnemonic. The operation names
now directly correspond to the builtin function names except for `
https://github.com/HighW4y2H3ll updated
https://github.com/llvm/llvm-project/pull/145652
>From 9a29dd6fa3f28bf507f047a22597f8510bd096b3 Mon Sep 17 00:00:00 2001
From: h2h
Date: Tue, 24 Jun 2025 23:24:32 -0700
Subject: [PATCH 1/9] [Clang] Allow vanilla C function symbol name to be used
in __att
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` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/186/builds/10643
Here is the relev
Sirraide wrote:
> my worry about ill-effects from canonicalization of the user's
> build-dir-relative file paths into absolute paths is a more general one
Yeah, and moreover, another point I just thought of is that users tend to have
more control over their own include directories, i.e. if you
https://github.com/carlosgalvezp approved this pull request.
https://github.com/llvm/llvm-project/pull/148357
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/carlosgalvezp approved this pull request.
https://github.com/llvm/llvm-project/pull/148352
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/148338
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/148195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/148359
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-with-thin-lto-ubuntu` running on `as-worker-92` while building `clang`
at step 7 "test-stage1-compiler".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/127/builds/4036
Here is the relevant pie
vbvictor wrote:
Build errors seem unrelated to this change
https://github.com/llvm/llvm-project/pull/147197
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-m68k-linux-cross`
running on `suse-gary-m68k-cross` while building `clang-tools-extra,clang` at
step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/27/builds/12990
Here is t
https://github.com/one-d-wide updated
https://github.com/llvm/llvm-project/pull/148018
>From b35f9ebdd2e345449c2cea8df8d6731cde33dcb3 Mon Sep 17 00:00:00 2001
From: "Remy D. Farley"
Date: Sat, 12 Jul 2025 11:53:58 +
Subject: [PATCH] [clang-query] Allow for trailing comma in matchers
---
c
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Baranov Victor (vbvictor)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/148352.diff
1 Files Affected:
- (modified) clang-tools-extra/docs/ReleaseNotes.rst (+2-2)
``diff
diff --git a/clang-tools
https://github.com/vbvictor created
https://github.com/llvm/llvm-project/pull/148352
None
>From faae7788f2789634796d065c8497e8f408c6b8e4 Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Sat, 12 Jul 2025 13:13:48 +0300
Subject: [PATCH] [clang-tidy][NFC] fixed invalid formatting in
ReleaseNo
https://github.com/vbvictor created
https://github.com/llvm/llvm-project/pull/148357
When having this code:
```cpp
namespace {
class MyClassOutOfAnon {
public:
MyClassOutOfAnon();
} // namespace
MyClassOutOfAnon::MyClassOutOfAnon() {}
```
`MyClassOutOfAnon::MyClassOutOfAnon` is located in ano
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Baranov Victor (vbvictor)
Changes
When having this code:
```cpp
namespace {
class MyClassOutOfAnon {
public:
MyClassOutOfAnon();
} // namespace
MyClassOutOfAnon::MyClassOutOfAnon() {}
```
`MyClassOutOfAnon::MyClassOutOfAnon` is loca
@@ -21,6 +21,17 @@ AST_MATCHER(NamedDecl, isInMacro) {
AST_MATCHER(VarDecl, isLocalVariable) { return Node.isLocalVarDecl(); }
+AST_MATCHER(Decl, isLexicallyInAnonymousNamespace) {
+ for (const DeclContext *DC = Node.getLexicalDeclContext(); DC != nullptr;
+ DC = DC->g
Author: Victor Chernyakin
Date: 2025-07-12T14:12:02+03:00
New Revision: 4326f68acbb12d21bc0c139c9b7f0a74a9ec8b69
URL:
https://github.com/llvm/llvm-project/commit/4326f68acbb12d21bc0c139c9b7f0a74a9ec8b69
DIFF:
https://github.com/llvm/llvm-project/commit/4326f68acbb12d21bc0c139c9b7f0a74a9ec8b69.d
https://github.com/bv2k4 created
https://github.com/llvm/llvm-project/pull/148359
None
>From 45c4d188b8ca895cc780550ad00cb9c5cd450447 Mon Sep 17 00:00:00 2001
From: Bogdan Vetrenko
Date: Sat, 12 Jul 2025 14:10:00 +0300
Subject: [PATCH] [clang][NFC] Fix typos and grammar in comments
---
clang
https://github.com/vbvictor closed
https://github.com/llvm/llvm-project/pull/147197
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
@localspook Congratulations on having your first Pull Request (PR) merged into
the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a
problem with a bui
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Bogdan Vetrenko (bv2k4)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/148359.diff
1 Files Affected:
- (modified) clang/lib/AST/ASTDiagnostic.cpp (+6-6)
``diff
diff --git a/clang/lib/AST/ASTDiagnostic.cpp b
one-d-wide wrote:
I don't think this is much of an issue. Once you attempt to interpret a query
with trailing comma in it as C++ code, a compiler (or even an LSP) will usually
emit a (mostly) unambiguous error, which should be trivial to correct. And
that's it. Existing queries, including ones
https://github.com/Harald-R created
https://github.com/llvm/llvm-project/pull/148371
Following the discussions from
https://github.com/llvm/llvm-project/pull/140594, this PR updates the logic in
`IncludeInserter` to also use the resolved paths instead of the spellings when
filtering headers.
@@ -306,14 +306,14 @@ IncludeInserter::calculateIncludePath(const HeaderFile
&InsertedHeader,
return std::nullopt;
bool IsAngled = false;
for (auto &Filter : AngledHeaders) {
-if (Filter(Suggested)) {
+if (Filter(InsertedHeader.File)) {
Harald-
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/148025
>From 3658e227161dc3e2cf0a6d7f21d008df6509e9c2 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Thu, 10 Jul 2025 20:37:25 +0200
Subject: [PATCH 1/2] [CIR] Implement SubOp for ComplexType
---
clang/includ
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-arm-ubuntu` running
on `linaro-lldb-arm-ubuntu` while building `clang-tools-extra,clang,lldb` at
step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/18/builds/18898
Here is the releva
@@ -2786,6 +2786,11 @@ struct FormatStyle {
/// \version 3.7
std::vector ForEachMacros;
+ /// A vector of function-like macros whose invocations should be skipped by
+ /// ``RemoveParentheses``.
+ /// \version 21
+ std::vector FunctionLikeMacros;
owenc
@@ -927,14 +927,12 @@ BreakableLineCommentSection::BreakableLineCommentSection(
}
if (Lines[i].size() != IndentPrefix.size()) {
-PrefixSpaceChange[i] = FirstLineSpaceChange;
+assert(Lines[i].size() > IndentPrefix.size());
-if (SpacesInPref
https://github.com/vbvictor created
https://github.com/llvm/llvm-project/pull/148384
I'm not sure if we need to add label inside link or not, but `lists.llvm.org`
seems outdated for sure, WDYT?
>From 11662e5e11ff14667893282cd05c4fa7c4be706e Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date:
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Baranov Victor (vbvictor)
Changes
I'm not sure if we need to add label inside link or not, but `lists.llvm.org`
seems outdated for sure, WDYT?
---
Full diff: https://github.com/llvm/llvm-project/pull/148384.diff
1 Files Affe
EugeneZelenko wrote:
There are dedicated Discourse topics for Clangd and Clang-Tidy. May be they
should be mentioned explicitly?
https://github.com/llvm/llvm-project/pull/148384
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
Author: Baranov Victor
Date: 2025-07-12T20:05:05+03:00
New Revision: 45f7285d0ef35615cc9ba665ed54fdb4c6d1c711
URL:
https://github.com/llvm/llvm-project/commit/45f7285d0ef35615cc9ba665ed54fdb4c6d1c711
DIFF:
https://github.com/llvm/llvm-project/commit/45f7285d0ef35615cc9ba665ed54fdb4c6d1c711.diff
https://github.com/vbvictor closed
https://github.com/llvm/llvm-project/pull/148357
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor updated
https://github.com/llvm/llvm-project/pull/148384
>From 11662e5e11ff14667893282cd05c4fa7c4be706e Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Sat, 12 Jul 2025 19:51:24 +0300
Subject: [PATCH 1/2] [clang-tools-extra] fix link to code review in README.txt
vbvictor wrote:
> There are dedicated Discourse topics for Clangd and Clang-Tidy. May be they
> should be mentioned explicitly?
Added links to them as a list, do you think we need a new entry for each of the
link?
https://github.com/llvm/llvm-project/pull/148384
__
EugeneZelenko wrote:
> > There are dedicated Discourse topics for Clangd and Clang-Tidy. May be they
> > should be mentioned explicitly?
>
> Added links to them as a list, do you think we need a new entry for each of
> the link? IMO from reading the link name, user can understand where it poin
https://github.com/EugeneZelenko approved this pull request.
Looks OK for me, but please wait for other reviewers, I may miss something :-)
https://github.com/llvm/llvm-project/pull/148384
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
https://github.com/carlosgalvezp approved this pull request.
https://github.com/llvm/llvm-project/pull/148384
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
schenker wrote:
I tested 6040ba1eed8d4fe6c573a11568c15cc66e31d4a3 and found this fixit that
does not compile:
**Input**
```cpp
namespace
{
struct N
{
[[maybe_unused]] friend void operator<<(int&, const N&) {}
};
}
```
```console
clang-tidy -checks="-*,*constexpr*" -export-fixes fixes.yaml tes
Author: Kazu Hirata
Date: 2025-07-12T11:28:26-07:00
New Revision: 78c12d380c369b53416518776ae53c3ba168cf7b
URL:
https://github.com/llvm/llvm-project/commit/78c12d380c369b53416518776ae53c3ba168cf7b
DIFF:
https://github.com/llvm/llvm-project/commit/78c12d380c369b53416518776ae53c3ba168cf7b.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/148338
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -370,40 +497,128 @@ std::unique_ptr Paragraph::clone() const {
/// Choose a marker to delimit `Text` from a prioritized list of options.
/// This is more readable than escaping for plain-text.
-llvm::StringRef chooseMarker(llvm::ArrayRef Options,
-
@@ -370,40 +497,128 @@ std::unique_ptr Paragraph::clone() const {
/// Choose a marker to delimit `Text` from a prioritized list of options.
/// This is more readable than escaping for plain-text.
-llvm::StringRef chooseMarker(llvm::ArrayRef Options,
-
tcottin wrote:
Thanks for the review!
I pushed some fixes.
https://github.com/llvm/llvm-project/pull/140498
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -370,40 +497,128 @@ std::unique_ptr Paragraph::clone() const {
/// Choose a marker to delimit `Text` from a prioritized list of options.
/// This is more readable than escaping for plain-text.
-llvm::StringRef chooseMarker(llvm::ArrayRef Options,
-
https://github.com/localspook created
https://github.com/llvm/llvm-project/pull/148399
We're currently on Unicode 14.0.0. This PR updates it to Unicode 16.0.0.
>From e252ea0a1ac94e8324bb332037508fd3f9b61317 Mon Sep 17 00:00:00 2001
From: Victor Chernyakin
Date: Sat, 12 Jul 2025 12:46:01 -0700
@@ -1601,51 +1591,32 @@ std::optional
getBacktickQuoteRange(llvm::StringRef Line,
return Line.slice(Offset, Next + 1);
}
-void parseDocumentationLine(llvm::StringRef Line, markup::Paragraph &Out) {
+void parseDocumentationParagraph(llvm::StringRef Text, markup::Paragraph &O
vbvictor wrote:
Ran this check on `Poco`, `VTK` and `opencv` successfully, stack traces are
printed correctly.
https://github.com/llvm/llvm-project/pull/134375
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
https://github.com/5chmidti edited
https://github.com/llvm/llvm-project/pull/139474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
5chmidti wrote:
Please add more tests:
- one of the args being, e.g., `int`, since there should still be a warning
- also add those same tests to the C++ tests to cover that they are being
handled correctly in both languages
https://github.com/llvm/llvm-proje
Author: Jason Rice
Date: 2025-07-13T06:17:41+03:00
New Revision: 6f923134ddf4afc4266c4c32854d7cc2793c23a1
URL:
https://github.com/llvm/llvm-project/commit/6f923134ddf4afc4266c4c32854d7cc2793c23a1
DIFF:
https://github.com/llvm/llvm-project/commit/6f923134ddf4afc4266c4c32854d7cc2793c23a1.diff
LO
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/121225
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/121225
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,
=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?Message-ID:
In-Reply-To:
@@ -9,6 +9,9 @@ different from the number of data members inside the union.
A struct or a class is conside
=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,
=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?Message-ID:
In-Reply-To:
@@ -147,3 +148,16 @@ struct Name {\
// CHECK-MESSAGES: :[[@LINE+1]]:44: warning: tagged union has more d
=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,
=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=,=?utf-8?b?R8OhYm9yIFTDs3RodsOhcmk=?=
Message-ID:
In-Reply-To:
@@ -140,6 +140,12 @@ Changes in existing checks
calls of ``std::string`` constructor with char pointer
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/148345
This allows RemoveParentheses to skip the invocations of function-like macros.
Fixes #68354.
Fixes #147780.
>From 3affefb96efe2d2955be66c247276b81d5d1d3a1 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Sat, 12 J
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
This allows RemoveParentheses to skip the invocations of function-like macros.
Fixes #68354.
Fixes #147780.
---
Full diff: https://github.com/llvm/llvm-project/pull/148345.diff
11 Files Affected:
- (mod
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-aarch64-linux-bootstrap-hwasan` running on `sanitizer-buildbot11`
while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/55/builds/14134
Here is the re
https://github.com/4vtomat updated
https://github.com/llvm/llvm-project/pull/145489
>From e51b061b7231ccb990e74a313f7cea900faf34c5 Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Sun, 6 Jul 2025 00:42:02 -0700
Subject: [PATCH] [clang][RISCV] Fix crash on VLS calling convention
This patch handl
https://github.com/zygoloid approved this pull request.
https://github.com/llvm/llvm-project/pull/148336
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
chestnykh wrote:
The buildbot fail looks unrelated
https://github.com/llvm/llvm-project/pull/147905
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zygoloid approved this pull request.
https://github.com/llvm/llvm-project/pull/148337
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2786,6 +2786,11 @@ struct FormatStyle {
/// \version 3.7
std::vector ForEachMacros;
+ /// A vector of function-like macros whose invocations should be skipped by
+ /// ``RemoveParentheses``.
+ /// \version 21
+ std::vector FunctionLikeMacros;
Hazar
@@ -927,14 +927,12 @@ BreakableLineCommentSection::BreakableLineCommentSection(
}
if (Lines[i].size() != IndentPrefix.size()) {
-PrefixSpaceChange[i] = FirstLineSpaceChange;
+assert(Lines[i].size() > IndentPrefix.size());
-if (SpacesInPref
localspook wrote:
Taken from: https://www.unicode.org/Public/security/16.0.0/confusables.txt
https://github.com/llvm/llvm-project/pull/148399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
https://github.com/5chmidti edited
https://github.com/llvm/llvm-project/pull/139474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vbvictor wrote:
Is this file taken as-is from some place or crafted by hand? Could you share a
source?
https://github.com/llvm/llvm-project/pull/148399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
@@ -352,40 +606,59 @@ TEST(BulletList, Render) {
DeepDoc.addParagraph().appendText("baz");
StringRef ExpectedMarkdown = R"md(- foo
- bar
-- foo
- baz
- - foo
-- baz
+- foo
+
tcottin wrote:
Yes.
We have to add them because we now end a bullet
@@ -0,0 +1,43 @@
+// REQUIRES: lld
+
+/// Check DTLTO options are forwarded to the linker.
+
+// RUN: echo "--target=x86_64-linux-gnu \
+// RUN: -Xthinlto-distributor=distarg1 \
+// RUN: -Xthinlto-distributor=distarg2,distarg3 \
+// RUN: -fuse-ld=lld" > %t.rsp
+
+/// Check t
@@ -403,6 +403,11 @@
// MCPU-MARCH: "-nostdsysteminc" "-target-cpu" "sifive-e31" "-target-feature"
"+m" "-target-feature" "+c"
// MCPU-MARCH: "-target-abi" "ilp32"
+// march=unset erases previous march
MaskRay wrote:
`/// -march=unset erases previous -march`
https://github.com/MaskRay approved this pull request.
Thanks! The reactions include a thumbs-up and a love. Setting my approval.
https://github.com/llvm/llvm-project/pull/148321
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
https://github.com/localspook edited
https://github.com/llvm/llvm-project/pull/148399
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc updated
https://github.com/llvm/llvm-project/pull/148321
>From 9521bd783966635f3219ac02e3fe43ed33294546 Mon Sep 17 00:00:00 2001
From: Craig Topper
Date: Fri, 11 Jul 2025 17:42:49 -0700
Subject: [PATCH 1/3] [RISCV] Add -march=unset to cancel and ignore a previous
-ma
@@ -91,7 +93,8 @@ void UseIntegerSignComparisonCheck::storeOptions(
void UseIntegerSignComparisonCheck::registerMatchers(MatchFinder *Finder) {
const auto SignedIntCastExpr = intCastExpression(true, "sIntCastExpression");
- const auto UnSignedIntCastExpr = intCastExpression
https://github.com/vbvictor commented:
Could you add more tests, e.g. when we have "inverted" casts in GH127471 like
this:
```cpp
(unsigned int)a2.size() == size() //size() return int
```
I believe we could also be add more tests with different type of casts in
GH127471 case.
https://github.c
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/144240
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -39,21 +39,23 @@ intCastExpression(bool IsSigned,
// std::cmp_{} functions trigger a compile-time error if either LHS or RHS
// is a non-integer type, char, enum or bool
// (unsigned char/ signed char are Ok and can be used).
- auto IntTypeExpr = expr(hasType(hasCanon
1 - 100 of 117 matches
Mail list logo
