https://github.com/Michael137 updated
https://github.com/llvm/llvm-project/pull/149827
>From bb71b69ea69fe4046a3f93e30f82dfb1d4d59b69 Mon Sep 17 00:00:00 2001
From: Michael Buch
Date: Mon, 18 Aug 2025 15:14:40 +0100
Subject: [PATCH 1/3] [clang][DebugInfo] Emit unified (Itanium) mangled name to
Author: Simon Pilgrim
Date: 2025-09-01T15:59:53Z
New Revision: e956090f7455750c7ce932801609c22a97b19a05
URL:
https://github.com/llvm/llvm-project/commit/e956090f7455750c7ce932801609c22a97b19a05
DIFF:
https://github.com/llvm/llvm-project/commit/e956090f7455750c7ce932801609c22a97b19a05.diff
LOG:
5chmidti wrote:
I'll have very little access to my machine this month, but I'll move this
forward in October (maybe with a bit before that) and also split the PR into
smaller ones.
https://github.com/llvm/llvm-project/pull/146553
___
cfe-commits mail
balazske wrote:
According to https://discourse.llvm.org/t/remove-alpha-core-castsize/87974 this
checker looks to be rarely used. In the current form it can have more false
positives (I tested it on some C projects) than useful results. Additionally,
the same problem could be found with other m
mizvekov wrote:
The problem is not whether the template instantiation is a definition or not,
if the type is required to be complete, then it needs to be a definition. If
it's not required to be complete, then it doesn't need to be.
What happens here is that when the instantiation is needed, b
@@ -3632,6 +3632,36 @@ static unsigned maxNestingDepth(const AnnotatedLine
&Line) {
return Result;
}
+// Returns the token after the first qualifier of the name, or nullptr if there
+// is no qualifier.
+static FormatToken *skipNameQualifier(const FormatToken *Tok) {
+ //
@@ -3632,6 +3632,36 @@ static unsigned maxNestingDepth(const AnnotatedLine
&Line) {
return Result;
}
+// Returns the token after the first qualifier of the name, or nullptr if there
+// is no qualifier.
+static FormatToken *skipNameQualifier(const FormatToken *Tok) {
+ //
yronglin wrote:
> Some nits but I think it looks good overall @erichkeane @Bigcheese
Thanks for your review!
https://github.com/llvm/llvm-project/pull/107168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/eulerkochy converted_to_draft
https://github.com/llvm/llvm-project/pull/156369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/eulerkochy edited
https://github.com/llvm/llvm-project/pull/156369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1119,43 +1115,158 @@ bool Preprocessor::LexHeaderName(Token &FilenameTok,
bool AllowMacroExpansion) {
return false;
}
+// We represent the primary and partition names as 'Paths' which are sections
+// of the hierarchical access path for a clang module. However for C++2
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/155612
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/melver closed
https://github.com/llvm/llvm-project/pull/155687
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Marco Elver
Date: 2025-09-01T21:50:13+02:00
New Revision: 5176fb8b1369f9738731ea3452828b29227e1e06
URL:
https://github.com/llvm/llvm-project/commit/5176fb8b1369f9738731ea3452828b29227e1e06
DIFF:
https://github.com/llvm/llvm-project/commit/5176fb8b1369f9738731ea3452828b29227e1e06.diff
L
@@ -3669,12 +3718,11 @@ static FormatToken *getFunctionName(const AnnotatedLine
&Line,
}
// Skip to the unqualified part of the name.
-while (Tok->startsSequence(tok::identifier, tok::coloncolon)) {
- assert(Tok->Next);
- Tok = Tok->Next->Next;
- if
@@ -3669,12 +3718,11 @@ static FormatToken *getFunctionName(const AnnotatedLine
&Line,
}
// Skip to the unqualified part of the name.
-while (Tok->startsSequence(tok::identifier, tok::coloncolon)) {
- assert(Tok->Next);
- Tok = Tok->Next->Next;
- if
https://github.com/paschalis-mpeis approved this pull request.
Thanks for the fixes. Directory `/bolt` LGTM. Please wait on other areas.
https://github.com/llvm/llvm-project/pull/156270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
https://github.com/NagyDonat approved this pull request.
I support the removal of this checker, because it implements a heuristic that
finds and reports a very specific bug which is (as far as I see) not a
"natural" mistake that would appear during real-world development.
https://github.com/ll
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/155612
>From e2125e4c9a9d8f3b45ba28af3a54cf13626a910f Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Wed, 27 Aug 2025 14:28:22 +0200
Subject: [PATCH] [Clang] Introduce __builtin_meow_synthesises_from_spaceship
@@ -3632,6 +3632,36 @@ static unsigned maxNestingDepth(const AnnotatedLine
&Line) {
return Result;
}
+// Returns the token after the first qualifier of the name, or nullptr if there
+// is no qualifier.
+static FormatToken *skipNameQualifier(const FormatToken *Tok) {
+ //
https://github.com/DaveBrantonCTCT updated
https://github.com/llvm/llvm-project/pull/155200
>From 19b4ddf6a856fdbebac19d1779c6ee83f2d682b0 Mon Sep 17 00:00:00 2001
From: Dave Branton
Date: Tue, 2 Sep 2025 08:59:08 +1200
Subject: [PATCH] Correct replacement code when signed stdint types are used
@@ -39,6 +39,55 @@ AST_MATCHER(clang::ParmVarDecl, isArgvOfMain) {
return FD ? FD->isMain() : false;
}
+template
+const TargetType *getAs(const NodeType *Node) {
+ if constexpr (std::is_same_v)
+return Node->template get();
+ else
+return llvm::dyn_cast(Node);
+}
SunBlack wrote:
I came across it because of #156058. I think the conversion can already be
useful now, but restrictions could be imposed.
> `std::span` has a capability gap with a const reference to a `std::vector`
> until C++26 as `std::span` cannot be constructed from an initializer list.
>
HighCommander4 wrote:
Changes to `clang-tools-extra/clangd` LGTM.
https://github.com/llvm/llvm-project/pull/156270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/BaLiKfromUA updated
https://github.com/llvm/llvm-project/pull/144313
>From b2e5ee18bf7f03bbf5f99ef124ef707dc3116886 Mon Sep 17 00:00:00 2001
From: Valentyn Yukhymenko
Date: Mon, 16 Jun 2025 08:55:06 +0100
Subject: [PATCH 1/4] first iteration of fix
---
.../bde/types/bdlb_nu
https://github.com/vidur2 closed
https://github.com/llvm/llvm-project/pull/155131
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aganea wrote:
Ping.
This still occurs on Windows as of today on main, `clang-cl -print-runtime-dir`
doesn't print the right folder path.
On disk we have: `C:\Program Files\LLVM\lib\clang\22\lib\windows`,
whereas `-print-runtime-path` prints the full target as on the other platforms:
`C:\Progra
https://github.com/DaveBrantonCTCT updated
https://github.com/llvm/llvm-project/pull/155200
>From 19b4ddf6a856fdbebac19d1779c6ee83f2d682b0 Mon Sep 17 00:00:00 2001
From: Dave Branton
Date: Tue, 2 Sep 2025 08:59:08 +1200
Subject: [PATCH 1/3] Correct replacement code when signed stdint types are
haoNoQ wrote:
> > At the moment of the previous PR, there was a concern about configurability
> > of check
>
> I think I was about adding an option to configure list of optional classes.
> Probably `OptionalClasses` or similar. I haven't seen issues about it, so it
> may not be a high priorit
https://github.com/eulerkochy created
https://github.com/llvm/llvm-project/pull/156369
None
>From e3795502f1acf7ec5c3980ed9fdb502bbf9403ba Mon Sep 17 00:00:00 2001
From: Koustav Chowdhury
Date: Tue, 2 Sep 2025 01:11:34 +0530
Subject: [PATCH] feat: constexpr SSE multiplication intrinsics
---
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Qiongsi Wu (qiongsiwu)
Changes
https://github.com/llvm/llvm-project/pull/148959 added two checks that
restricts the uses of link declarations. It is later discovered that the Swift
project uses link declarations in submodules extensively
MythreyaK wrote:
Glad to help, Thank you for the review!
https://github.com/llvm/llvm-project/pull/155561
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yingcong-wu updated
https://github.com/llvm/llvm-project/pull/156383
>From b69df27e57cc0b0673960bc6802693df9fcc1e03 Mon Sep 17 00:00:00 2001
From: "Wu, Yingcong"
Date: Tue, 2 Sep 2025 09:42:07 +0800
Subject: [PATCH 1/4] switch to llvm-objdump
---
libunwind/test/eh_frame_fde
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kito Cheng (kito-cheng)
Changes
For each intrinsic with ManualCodegen block will generate something like below:
```cpp
SegInstSEW = 0;
...
if (SegInstSEW == (unsigned)-1) {
auto PointeeType =
E->getArg(4294967295)->getType()->ge
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-aarch64-linux-bootstrap-msan` running on `sanitizer-buildbot10`
while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/94/builds/10425
Here is the rele
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/16532
Here is the re
https://github.com/vedantjk created
https://github.com/llvm/llvm-project/pull/156380
Enable constexpr usage of vector bitcast intrinsics by changing from
`__DEFAULT_FN_ATTRS*` to
`__DEFAULT_FN_ATTRS*_CONSTEXPR` attributes.
**Updated intrinsics:**
- AVX-512 bitcast intrinsics: `_mm512_cas
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/zyn0217 commented:
I think we need a release note, maybe 'Potential AST breaking changes' would be
a good fit.
https://github.com/llvm/llvm-project/pull/156329
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.ll
https://github.com/wenju-he created
https://github.com/llvm/llvm-project/pull/156378
_CLC_DEF_WEAK can be used in our downstream libclc to allow overriding generic
__clc_tgamma implementation.
>From cc51cd1e096794162a5f3c7be9aa160d83ba2547 Mon Sep 17 00:00:00 2001
From: Wenju He
Date: Tue, 2
jj-marr wrote:
@SunBlack I'm still in favour of the check but I didn't see myself working on
it until C++26 as it won't be usable for many people.
I would prefer Option 2 if I was still working on this. In the example you
gave, the initializer list should be in a constexpr std::array to avoid
https://github.com/yingcong-wu edited
https://github.com/llvm/llvm-project/pull/156383
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yingcong-wu edited
https://github.com/llvm/llvm-project/pull/156383
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,15 @@
+// This test checks that under implicit modules, different optimization levels
ChuanqiXu9 wrote:
I feel this was caused by a typo. @jansvoboda11 can you fix this?
https://github.com/llvm/llvm-project/pull/146422
yingcong-wu wrote:
But I have added `// REQUIRES: linux` to the test already, why is it still
running on win-x-aarch64 ? Do you have any ideas? @philnik777
https://github.com/llvm/llvm-project/pull/154902
___
cfe-commits mailing list
cfe-commits@list
@@ -150,6 +150,9 @@ Changes to the Debug Info
Changes to the LLVM tools
-
+* modernize-use-std-format now correctly replaces signed types, and correctly
DaveBrantonCTCT wrote:
Thank you, have put this in the right place.
http
https://github.com/qiongsiwu updated
https://github.com/llvm/llvm-project/pull/156377
>From 0cb749e2ec11cf51d2c97e935727e55aeaf65097 Mon Sep 17 00:00:00 2001
From: Qiongsi Wu
Date: Mon, 1 Sep 2025 16:09:21 -0700
Subject: [PATCH] Permit link decls in submodule declarations.
---
clang/include/c
llvmbot wrote:
@llvm/pr-subscribers-libunwind
Author: Wu Yingcong (yingcong-wu)
Changes
In https://github.com/llvm/llvm-project/pull/154902, the test failed with
llvm-clang-win-x-aarch64(it pretends to be a Linux build?), and objdump is not
available there.
Change the test to use llvm-o
@@ -249,6 +249,10 @@ Changes in existing checks
` check to recognize
literal suffixes added in C++23 and C23.
+- Improved :doc:`modernize-use-std-format
EugeneZelenko wrote:
Please keep alphabetcial order (by check name) in this list.
https://github.com
@@ -249,6 +249,10 @@ Changes in existing checks
` check to recognize
literal suffixes added in C++23 and C23.
+- Improved :doc:`modernize-use-std-format
+ ` now correctly replaces signed
types, and correctly
EugeneZelenko wrote:
Please follow 80 chara
https://github.com/qiongsiwu created
https://github.com/llvm/llvm-project/pull/156377
https://github.com/llvm/llvm-project/pull/148959 added two checks that
restricts the uses of link declarations. It is later discovered that the Swift
project uses link declarations in submodules extensively o
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/vedantjk updated
https://github.com/llvm/llvm-project/pull/156381
>From 9833270e2861fe77e7e8e862d81bacbd39983895 Mon Sep 17 00:00:00 2001
From: vedantjk
Date: Mon, 1 Sep 2025 18:02:01 -0700
Subject: [PATCH] [clang] Make vector cast intrinsics constexpr
---
clang/lib/Headers
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina ,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina ,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina ,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina ,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina ,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina ,
=?utf-8?q?Tom=C3=A1=C5=A1?= Slanina
Message-ID:
In-Reply-To:
owenca
yingcong-wu wrote:
> @yingcong-wu The new test fails on the Arm bots. Please fix it soon or revert
> to get the precommit CI green again (finally).
Sure, I will take a quick look now.
https://github.com/llvm/llvm-project/pull/154902
___
cfe-commits m
@@ -213,6 +213,11 @@ Changes in existing checks
when the format string is converted to a different type by an implicit
constructor call.
+- Improved :doc:`modernize-use-std-format
EugeneZelenko wrote:
Sorry, I missed existing `modernize-use-std-format` e
https://github.com/DaveBrantonCTCT updated
https://github.com/llvm/llvm-project/pull/155200
>From 19b4ddf6a856fdbebac19d1779c6ee83f2d682b0 Mon Sep 17 00:00:00 2001
From: Dave Branton
Date: Tue, 2 Sep 2025 08:59:08 +1200
Subject: [PATCH 1/5] Correct replacement code when signed stdint types are
https://github.com/ckoparkar edited
https://github.com/llvm/llvm-project/pull/156381
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -113,93 +113,93 @@ _mm512_set1_pch(_Float16 _Complex __h) {
return (__m512h)_mm512_set1_ps(__builtin_bit_cast(float, __h));
}
-static __inline__ __m128 __DEFAULT_FN_ATTRS128 _mm_castph_ps(__m128h __a) {
+static __inline__ __m128 __DEFAULT_FN_ATTRS128_CONSTEXPR _mm_castph_
@@ -0,0 +1,34 @@
+# RUN: not llvm-mc -triple=riscv32 --mattr=+experimental-zibi %s 2>&1 \
+# RUN:| FileCheck %s --check-prefix=CHECK-ERROR
+# RUN: not llvm-mc -triple=riscv64 --mattr=+experimental-zibi %s 2>&1 \
+# RUN:| FileCheck %s --check-prefix=CHECK-ERROR
+beq
https://github.com/wangpc-pp approved this pull request.
https://github.com/llvm/llvm-project/pull/156397
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nikic wrote:
```
Building CXX object
tools/clang/tools/extra/clang-reorder-fields/CMakeFiles/obj.clangReorderFields.dir/Designator.cpp.o
FAILED:
tools/clang/tools/extra/clang-reorder-fields/CMakeFiles/obj.clangReorderFields.dir/Designator.cpp.o
sccache /opt/llvm/bin/clang++ -DGTEST_HAS_RTTI=0
@@ -0,0 +1,132 @@
+//===--===//
+//
+// 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: Apa
@@ -0,0 +1,132 @@
+//===--===//
+//
+// 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: Apa
https://github.com/t-a-james updated
https://github.com/llvm/llvm-project/pull/154746
>From 39ad4945d05f3e88fd32dfdea585aa99c6ba985f Mon Sep 17 00:00:00 2001
From: Tom James
Date: Thu, 21 Aug 2025 13:14:24 +0100
Subject: [PATCH 01/23] [clang-tidy] New bugprone-method-hiding check
---
.../bugp
Author: Nikita Popov
Date: 2025-09-01T10:07:11+02:00
New Revision: c128b8c46f2a3b750c9abcba1e303f92d6531e5f
URL:
https://github.com/llvm/llvm-project/commit/c128b8c46f2a3b750c9abcba1e303f92d6531e5f
DIFF:
https://github.com/llvm/llvm-project/commit/c128b8c46f2a3b750c9abcba1e303f92d6531e5f.diff
@@ -0,0 +1,127 @@
+//===--===//
+//
+// 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: Apa
@@ -0,0 +1,132 @@
+//===--===//
+//
+// 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: Apa
https://github.com/frederic-tingaud-sonarsource updated
https://github.com/llvm/llvm-project/pull/156011
>From 56c46c1224927289e08b8627484eab51dc21a69b Mon Sep 17 00:00:00 2001
From: Fred Tingaud
Date: Mon, 25 Aug 2025 20:07:04 +0200
Subject: [PATCH] [clang][AST] Fix source range of class templ
@@ -237,85 +238,85 @@ unsigned long long test_blsr_u64(unsigned long long __X) {
// Test constexpr handling.
#if defined(__cplusplus) && (__cplusplus >= 201103L)
-char andnu32[__andn_u32(0x01234567, 0xFECDBA98) == (~0x01234567 & 0xFECDBA98)
? 1 : -1];
-char andn2u32[_andn_u32
@@ -204,6 +205,10 @@ static cl::opt
EnableLoopInterchange("enable-loopinterchange", cl::init(false),
cl::Hidden,
cl::desc("Enable the LoopInterchange Pass"));
+static cl::opt EnableLoopFusion("enable-loopfusion", cl::init(false),
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Brad Smith (brad0)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/156299.diff
3 Files Affected:
- (modified) clang/lib/Driver/ToolChains/Haiku.h (+5)
- (modified) clang/lib/Driver/ToolChains/Managarm.h (+6)
- (modif
nikic wrote:
Fixed by
https://github.com/llvm/llvm-project/commit/c128b8c46f2a3b750c9abcba1e303f92d6531e5f.
https://github.com/llvm/llvm-project/pull/142150
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
https://github.com/Sockke closed
https://github.com/llvm/llvm-project/pull/154732
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
atrosinenko wrote:
@labrinea I'm sorry for disappearing, I will update this PR this week.
https://github.com/llvm/llvm-project/pull/141573
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
https://github.com/Keenuts approved this pull request.
Looks ok to me. So far seems like even on main, setting `-DLLVM_LIBDIR_SUFFIX`
has no effect so seems like this is ok
Adding @kwk in case I'm missing something
https://github.com/llvm/llvm-project/pull/155844
___
https://github.com/Keenuts created
https://github.com/llvm/llvm-project/pull/156338
Some testing was lacking, as well as DirectX lowering of the intrinsic.
Fixes #99159
From fd5bfbba2de4ae25336487b9778d56b5292d60ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nathan=20Gau=C3=ABr?=
Date: Mon, 1 Se
https://github.com/RKSimon closed
https://github.com/llvm/llvm-project/pull/156337
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2485,11 +2536,12 @@ class Preprocessor {
}
/// If we're importing a standard C++20 Named Modules.
- bool isInImportingCXXNamedModules() const {
-// NamedModuleImportPath will be non-empty only if we're importing
-// Standard C++ named modules.
-return !Nam
@@ -233,6 +233,9 @@ class Token {
PtrData = const_cast(Ptr);
}
+ template T getAnnotationValueAs() const {
+return static_cast(getAnnotationValue());
+ }
yronglin wrote:
It's unecessary, removed.
https://github.com/llvm/llvm-project/pull/107168
carlosgalvezp wrote:
> we shouldn't discourage people from improving the codebase
These are orthogonal issues. People are always welcome to improve the codebase,
but small PRs are essential for a healthy codebase. And it will actually be
**less* work for the author, because the patches will be
NagyDonat wrote:
> So is it best to close this PR?
Yes, I think this PR should be closed (because although it is a good approach
for fixing the bug, we have that other commit which is further along the same
path).
https://github.com/llvm/llvm-project/pull/155131
__
https://github.com/naveen-seth updated
https://github.com/llvm/llvm-project/pull/156248
>From 9760106dbd916861d396b90cc30262a13a01c9c2 Mon Sep 17 00:00:00 2001
From: Naveen Seth Hanig
Date: Mon, 25 Aug 2025 02:36:42 +0200
Subject: [PATCH 1/4] [clang][modules-driver] Move logic to enable
-fmodu
5chmidti wrote:
I won't have access to my machine for 2 weeks, so I can't try it out in the
coming days, sorry. I'll see if I can do at least some review on mobile, but my
general thoughts are in discourse.
For now: I think the documentation should be extended to explain that the AST
is not a
https://github.com/tblah approved this pull request.
Flang and flang-rt changes LGTM.
https://github.com/llvm/llvm-project/pull/156270
___
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/152047
>From bb17fb65d031190efff6a6ec1c0ca6d9c0d35abb Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Tue, 5 Aug 2025 01:15:02 +0300
Subject: [PATCH 1/7] [clang-tidy] Add new check 'llvm-use-ranges'
---
.../clan
https://github.com/balazske created
https://github.com/llvm/llvm-project/pull/156350
None
From 19c4f21ce4b1c79c8dd2be0a017164e500ac1b46 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Fri, 29 Aug 2025 09:02:39 +0200
Subject: [PATCH] [clang][analyzer] Remove checker 'al
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balázs Kéri (balazske)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/156350.diff
11 Files Affected:
- (modified) clang/docs/analyzer/checkers.rst (-12)
- (modified) clang
Author: moorabbit
Date: 2025-09-01T16:04:17Z
New Revision: e932e413cfd00d42b2832c7d5fc4b3db576a1401
URL:
https://github.com/llvm/llvm-project/commit/e932e413cfd00d42b2832c7d5fc4b3db576a1401
DIFF:
https://github.com/llvm/llvm-project/commit/e932e413cfd00d42b2832c7d5fc4b3db576a1401.diff
LOG: [He
https://github.com/RKSimon closed
https://github.com/llvm/llvm-project/pull/156187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RKSimon auto_merge_enabled
https://github.com/llvm/llvm-project/pull/156337
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -728,23 +678,15 @@ void
CallAndMessageChecker::HandleNilReceiver(CheckerContext &C,
C.addTransition(state);
}
-void ento::registerCallAndMessageModeling(CheckerManager &mgr) {
- mgr.registerChecker();
-}
-
-bool ento::shouldRegisterCallAndMessageModeling(const CheckerMa
mizvekov wrote:
> OK, I understand. Could you point me to where that pattern selection happens?
> I will not be able to work on that PR for two weeks but I can go back on it
> afterward.
In `Sema::InstantiateClass`, which is implemented in
SemaTemplateInstantiate.cpp.
https://github.com/llvm
https://github.com/bolshakov-a created
https://github.com/llvm/llvm-project/pull/156329
`VarTemplateSpecializationDecl::getTemplateArgsAsWritten()` function should
return `nullptr` in the case of implicit instantiation, as its
`ClassTemplateSpecializationDecl` counterpart does, and not the arg
@@ -1776,6 +1801,22 @@ class Preprocessor {
/// Lex the parameters for an #embed directive, returns nullopt on error.
std::optional LexEmbedParameters(Token &Current,
bool ForHasEmbed);
+ bool LexModuleNameContin
@@ -1178,186 +1289,39 @@ bool Preprocessor::LexAfterModuleImport(Token &Result)
{
// Figure out what kind of lexer we actually have.
recomputeCurLexerKind();
- // Lex the next token. The header-name lexing rules are used at the start of
- // a pp-import.
- //
- // For
@@ -1119,43 +1115,158 @@ bool Preprocessor::LexHeaderName(Token &FilenameTok,
bool AllowMacroExpansion) {
return false;
}
+// We represent the primary and partition names as 'Paths' which are sections
+// of the hierarchical access path for a clang module. However for C++2
@@ -2485,11 +2536,12 @@ class Preprocessor {
}
/// If we're importing a standard C++20 Named Modules.
- bool isInImportingCXXNamedModules() const {
-// NamedModuleImportPath will be non-empty only if we're importing
-// Standard C++ named modules.
-return !Nam
@@ -14,7 +14,9 @@
#include "clang/AST/ASTConsumer.h"
#include "clang/AST/ASTMutationListener.h"
#include "clang/AST/DynamicRecursiveASTVisitor.h"
+#include "clang/Basic/SourceLocation.h"
#include "clang/Lex/HeaderSearch.h"
+#include "clang/Lex/ModuleLoader.h"
https://github.com/RKSimon closed
https://github.com/llvm/llvm-project/pull/156327
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 224 matches
Mail list logo
