github-actions[bot] wrote:
# :window: Windows x64 Test Results
* 2995 tests passed
* 30 tests skipped
* 1 test failed
## Failed Tests
(click on a test name to see its output)
### Clangd Unit Tests
Clangd Unit
Tests._/ClangdTests_exe/BackgroundIndexTest/ConstructorForwarding
```
Script:
--
timon-ul wrote:
Oh that is very appreciated, I kinda had assumed that we have no way of testing
this, but that makes 0 sense, I should have asked about this possibility, my
bad.
https://github.com/llvm/llvm-project/pull/169742
___
cfe-commits mailing
HighCommander4 wrote:
Haven't made progress on debugging this yet, but I wrote an additional unit
test which exercises the background index codepath, to allow debugging the
issue under more controlled conditions.
I added the new test to this PR.
https://github.com/llvm/llvm-project/pull/16974
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/169742
>From 44ebad6933fccab5bcfc866ee56dd5381da5f452 Mon Sep 17 00:00:00 2001
From: Timon Ulrich
Date: Mon, 10 Nov 2025 10:40:14 +0100
Subject: [PATCH 01/15] clangd: make forwarding heuristic available for more
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/172135
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AditiRM updated
https://github.com/llvm/llvm-project/pull/154715
>From 20acfd4109a453060f3678e536234de94012f19c Mon Sep 17 00:00:00 2001
From: AditiRM
Date: Thu, 21 Aug 2025 09:40:59 +
Subject: [PATCH 01/29] Implement frontend for bcd builtins
---
clang/include/clang/Ba
https://github.com/zeyi2 updated
https://github.com/llvm/llvm-project/pull/172123
>From 4b0b1ee150ea820f00564667e734ddd225665d53 Mon Sep 17 00:00:00 2001
From: mtx
Date: Fri, 12 Dec 2025 22:39:40 +0800
Subject: [PATCH 01/23] [Github][CI] Introduce `doc8` to `code-lint-helper.py`
---
.../githu
@@ -104,6 +104,35 @@ bool SemaPPC::CheckPPCBuiltinFunctionCall(const TargetInfo
&TI,
return Diag(TheCall->getBeginLoc(), diag::err_64_bit_builtin_32_bit_tgt)
<< TheCall->getSourceRange();
+ // Common BCD type-validation helpers
+ // Emit diagnostics and retur
https://github.com/AditiRM updated
https://github.com/llvm/llvm-project/pull/154715
>From 20acfd4109a453060f3678e536234de94012f19c Mon Sep 17 00:00:00 2001
From: AditiRM
Date: Thu, 21 Aug 2025 09:40:59 +
Subject: [PATCH 01/28] Implement frontend for bcd builtins
---
clang/include/clang/Ba
https://github.com/zeyi2 updated
https://github.com/llvm/llvm-project/pull/172123
>From 4b0b1ee150ea820f00564667e734ddd225665d53 Mon Sep 17 00:00:00 2001
From: mtx
Date: Fri, 12 Dec 2025 22:39:40 +0800
Subject: [PATCH 01/22] [Github][CI] Introduce `doc8` to `code-lint-helper.py`
---
.../githu
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,517 @@
+//==
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,517 @@
+//==
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,501 @@
+//==
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,517 @@
+//==
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,517 @@
+//==
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,517 @@
+//==
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?= ,
Stefan =?utf-8?q?Gränitz?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,517 @@
+//==
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?= ,
Stefan =?utf-8?q?Gr=C3=A4nitz?=
Message-ID:
In-Reply-To:
@@ -104,6 +104,35 @@ bool SemaPPC::CheckPPCBuiltinFunctionCall(const TargetInfo
&TI,
return Diag(TheCall->getBeginLoc(), diag::err_64_bit_builtin_32_bit_tgt)
<< TheCall->getSourceRange();
+ // Common BCD type-validation helpers
+ // Emit diagnostics and retur
https://github.com/AditiRM updated
https://github.com/llvm/llvm-project/pull/154715
>From 20acfd4109a453060f3678e536234de94012f19c Mon Sep 17 00:00:00 2001
From: AditiRM
Date: Thu, 21 Aug 2025 09:40:59 +
Subject: [PATCH 01/27] Implement frontend for bcd builtins
---
clang/include/clang/Ba
https://github.com/kashika0112 edited
https://github.com/llvm/llvm-project/pull/171972
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
davemgreen wrote:
The MVE side is now all committed, making sure that we don't scalarize the
intrinsics. There was a question about whether we should be using target
specific intrinsics for it, but I believe we need to as we can't really convert
the constrained intrinsics into mve instructions
@@ -160,11 +161,37 @@ class LifetimeChecker {
}
}
+ /// Returns the declaration of a function that is visible across translation
+ /// units, if such a declaration exists and is different from the definition.
+ static const FunctionDecl *getCrossTUDecl(const FunctionD
@@ -112,6 +141,29 @@ bool SemaPPC::CheckPPCBuiltinFunctionCall(const TargetInfo
&TI,
case PPC::BI__builtin_ppc_packed2zoned:
case PPC::BI__builtin_ppc_zoned2packed:
return SemaRef.BuiltinConstantArgRange(TheCall, 1, 0, 1);
+ case PPC::BI__builtin_ppc_bcdshift:
+ case
@@ -112,6 +141,29 @@ bool SemaPPC::CheckPPCBuiltinFunctionCall(const TargetInfo
&TI,
case PPC::BI__builtin_ppc_packed2zoned:
case PPC::BI__builtin_ppc_zoned2packed:
return SemaRef.BuiltinConstantArgRange(TheCall, 1, 0, 1);
+ case PPC::BI__builtin_ppc_bcdshift:
+ case
https://github.com/AditiRM updated
https://github.com/llvm/llvm-project/pull/154715
>From 20acfd4109a453060f3678e536234de94012f19c Mon Sep 17 00:00:00 2001
From: AditiRM
Date: Thu, 21 Aug 2025 09:40:59 +
Subject: [PATCH 01/26] Implement frontend for bcd builtins
---
clang/include/clang/Ba
https://github.com/topperc closed
https://github.com/llvm/llvm-project/pull/172095
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Craig Topper
Date: 2025-12-14T22:32:03-08:00
New Revision: 7fa062ad5869feb9bcbec25354709fed06ab853b
URL:
https://github.com/llvm/llvm-project/commit/7fa062ad5869feb9bcbec25354709fed06ab853b
DIFF:
https://github.com/llvm/llvm-project/commit/7fa062ad5869feb9bcbec25354709fed06ab853b.diff
MythreyaK wrote:
Rebased onto main (with no changes)
https://github.com/llvm/llvm-project/pull/170642
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MythreyaK updated
https://github.com/llvm/llvm-project/pull/170642
>From a9b1f8877759c84256c9eb0bb3a37cf342e3c193 Mon Sep 17 00:00:00 2001
From: Mythreya
Date: Wed, 3 Dec 2025 21:41:40 -0800
Subject: [PATCH 1/6] Enable designator hints test
---
.../clangd/unittests/InlayHin
zeyi2 wrote:
I plan to split the `Dockerfile` modification to another PR, in order to avoid
potential CI failures (e.g. If the Dockerfile modification is not shipped in
advance, after changing `image: 'ghcr.io/llvm/ci-ubuntu-24.04-lint-doc8'`
before merging, the CI here will fail in `code-lint
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `reverse-iteration` running
on `hexagon-build-03` while building `clang` at step 6 "check_all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/110/builds/6802
Here is the relevant piece of the build l
https://github.com/xlauko closed
https://github.com/llvm/llvm-project/pull/172153
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Henrich Lauko
Date: 2025-12-15T07:13:41+01:00
New Revision: 5a581acb297069a87e32854ca4853c415a509d5a
URL:
https://github.com/llvm/llvm-project/commit/5a581acb297069a87e32854ca4853c415a509d5a
DIFF:
https://github.com/llvm/llvm-project/commit/5a581acb297069a87e32854ca4853c415a509d5a.diff
https://github.com/localspook created
https://github.com/llvm/llvm-project/pull/172255
None
>From c1d870a8219554f8a31af9e79f1034b2107f94a6 Mon Sep 17 00:00:00 2001
From: Victor Chernyakin
Date: Sun, 14 Dec 2025 21:50:56 -0800
Subject: [PATCH] [clang-tidy][NFC] Refactor
`cppcoreguidelines-spec
kawashima-fj wrote:
LGTM. Thanks a lot.
I tried this patch on Linux/Ubuntu 24.04. OpenMP and Flang-RT are included in
the tarball as expected.
https://github.com/llvm/llvm-project/pull/160581
___
cfe-commits mailing list
[email protected]
htt
https://github.com/eugeneepshteyn updated
https://github.com/llvm/llvm-project/pull/172252
>From 509af02a62af56d688d2325bfffc73272933652c Mon Sep 17 00:00:00 2001
From: Eugene Epshteyn
Date: Mon, 15 Dec 2025 00:56:23 -0500
Subject: [PATCH 1/2] [flang] Support -funsafe-cray-pointers
If -funsafe
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 origin/main HEAD --extensions cpp --
clang/lib/Driver/ToolChains/Flang.cpp --diff_fro
kevinsala wrote:
Fixed by #172249 from the CUDA plugin.
https://github.com/llvm/llvm-project/pull/156229
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kevinsala closed
https://github.com/llvm/llvm-project/pull/156229
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/eugeneepshteyn created
https://github.com/llvm/llvm-project/pull/172252
If -funsafe-cray-pointers is passed to flang driver, pass '-mmlir
-funsafe-cray-pointers' to the compiler.
>From 509af02a62af56d688d2325bfffc73272933652c Mon Sep 17 00:00:00 2001
From: Eugene Epshteyn
D
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Younan Zhang (zyn0217)
Changes
In concept checking, we need to transform SubstNTTPExpr when evaluating
constraints.
The value category is initially computed during parameter mapping, possibly
with a dependent expression. However during i
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/172251
In concept checking, we need to transform SubstNTTPExpr when evaluating
constraints.
The value category is initially computed during parameter mapping, possibly
with a dependent expression. However during inst
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/169965
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1706,7 +1706,12 @@ CIRGenFunction::emitX86BuiltinExpr(unsigned builtinID,
const CallExpr *expr) {
cir::SyncScopeKind::SingleThread));
return mlir::Value{};
}
- case X86::BI_AddressOfReturnAddress:
+ case X86::BI_AddressOfReturnAd
https://github.com/4vtomat updated
https://github.com/llvm/llvm-project/pull/172135
>From 3e4c3c13eba70ff8b347bcc8c40cdb4ad0b80e75 Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Fri, 12 Dec 2025 23:08:16 -0800
Subject: [PATCH 1/2] [llvm][RISCV] Add frm range check for xsfvfnrclipxfqf
---
cla
https://github.com/4vtomat closed
https://github.com/llvm/llvm-project/pull/172130
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/4vtomat updated
https://github.com/llvm/llvm-project/pull/171981
>From 959b7f73fc6a7c542640256927571f767a3a4f68 Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Fri, 12 Dec 2025 01:15:38 -0800
Subject: [PATCH 1/4] [AST][RISCV] Preserve RISC-V intrinsic pragma in AST
RISC-V ve
@@ -123,10 +124,10 @@ custom reason.
::std::original(); // no-warning
original_function(); // no-warning
-If the regular expression contains the character ``:``, it is matched against
the
-qualified name (i.e. ``std::original``), otherwise the regex is matched against
-
@@ -96,21 +96,22 @@ to be checked. The format is the following, without
newlines:
The functions are matched using POSIX extended regular expressions.
*(Note: The regular expressions do not support negative* ``(?!)`` *matches.)*
-The ``reason`` is optional and is used to provi
@@ -123,10 +124,10 @@ custom reason.
::std::original(); // no-warning
original_function(); // no-warning
-If the regular expression contains the character ``:``, it is matched against
the
-qualified name (i.e. ``std::original``), otherwise the regex is matched against
-
@@ -96,21 +96,22 @@ to be checked. The format is the following, without
newlines:
The functions are matched using POSIX extended regular expressions.
*(Note: The regular expressions do not support negative* ``(?!)`` *matches.)*
-The ``reason`` is optional and is used to provi
zeyi2 wrote:
> Should be removed.
Fixed, sorry that I missed the testcases differences. Thanks!
https://github.com/llvm/llvm-project/pull/172220
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/c
https://github.com/zeyi2 edited https://github.com/llvm/llvm-project/pull/172220
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zwuis wrote:
> Closes #172180
Should be removed.
```cpp
if (bool b = ...; b == true) { ... }
// target in that issue
if (bool b = ...) { ... }
// what this PR does currently (from tests)
if (bool b = ...; b) { ... }
```
https://github.com/llvm/llvm-project/pull/172220
_
https://github.com/wenju-he edited
https://github.com/llvm/llvm-project/pull/171959
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Luhaocong edited
https://github.com/llvm/llvm-project/pull/171974
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5911,6 +5911,30 @@ AST_POLYMORPHIC_MATCHER_P(hasAnySubstatement,
Builder) != CS->body_end();
}
+/// Matches compound statements that contain adjacent substatements matching
+/// the provided sequence of matchers. Also matches StmtE
https://github.com/zeyi2 updated
https://github.com/llvm/llvm-project/pull/168722
>From 4908ebaa5652957761db6116a3e4c823877fd9e3 Mon Sep 17 00:00:00 2001
From: mtx
Date: Wed, 19 Nov 2025 23:19:32 +0800
Subject: [PATCH 1/3] [clang-tidy][NFC] Fix formatting issue in
`unsafe-functions.rst`
---
https://github.com/Luhaocong updated
https://github.com/llvm/llvm-project/pull/171974
>From 9772a1690ff6d40db9f1263b812f644b095d7e66 Mon Sep 17 00:00:00 2001
From: Haocong Lu
Date: Fri, 12 Dec 2025 13:55:59 +0800
Subject: [PATCH] [CIR][X86] Implement lowering for `_AddressOfReturnAddress`
buil
https://github.com/irishrover updated
https://github.com/llvm/llvm-project/pull/172170
>From 210ff50ef6a9b5185195970df6ad51587f8e1fc1 Mon Sep 17 00:00:00 2001
From: Zinovy Nis
Date: Sat, 13 Dec 2025 13:47:01 +0300
Subject: [PATCH] [clang-tidy] Add a new check 'replace-with-string-view'
Looks f
https://github.com/zwuis approved this pull request.
https://github.com/llvm/llvm-project/pull/169965
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -753,6 +753,7 @@ AST Matchers
- Added ``hasExplicitParameters`` for ``LambdaExpr`` as an output attribute to
AST JSON dumps.
- Add ``arrayTypeLoc`` matcher for matching ``ArrayTypeLoc``.
+- Add ``hasAnySubstatement``.
zwuis wrote:
Copy-paste error?
https
https://github.com/zwuis edited https://github.com/llvm/llvm-project/pull/169965
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
j39m wrote:
I've been waiting on the `build and test {linux,windows}` steps for a few days
now. Is there something I need to do to make them resolve?
I will still need a hint on what build target to use to repro the failure in my
tree.
https://github.com/llvm/llvm-project/pull/170822
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-ppc64le-linux-multistage` running on `ppc64le-clang-multistage-test`
while building `clang-tools-extra` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/76/builds/13571
https://github.com/wangpc-pp approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/172095
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ChuanqiXu9 wrote:
> For us, it's because at a certain scale with Thrift generated files, we end
> up importing 1K+ modules that all declare the apache::thrift namespace.
Maybe this is a nice point of named modules. We use thrift too. But we just
wrap it into a single named module. Then we avoi
@@ -768,6 +779,41 @@ struct GetReturnObjectManager {
CGF.EmitAutoVarInit(GroEmission);
Builder.CreateStore(Builder.getTrue(), GroActiveFlag);
}
+
+ void EmitGroConv() {
+// GRO conversion is unnecessary when get_return_object's type matches the
+// coroutine
@@ -950,15 +1035,20 @@ void CodeGenFunction::EmitCoroutineBody(const
CoroutineBodyStmt &S) {
// We don't need FinalBB. Emit it to make sure the block is deleted.
EmitBlock(FinalBB, /*IsFinished=*/true);
}
+
+// GRO conversion is unnecessary when get_return_
@@ -973,8 +1063,18 @@ void CodeGenFunction::EmitCoroutineBody(const
CoroutineBodyStmt &S) {
// shouldn't change the AST.
if (PreviousRetValue)
cast(Ret)->setRetValue(PreviousRetValue);
- }
+if (!GroManager.DirectEmit) {
+ EndCall->moveBefore(RetBB->ge
@@ -768,6 +781,76 @@ struct GetReturnObjectManager {
CGF.EmitAutoVarInit(GroEmission);
Builder.CreateStore(Builder.getTrue(), GroActiveFlag);
}
+ // The GRO returns either when it is first suspended or when it completes
+ // without ever being suspended. The EmitGro
@@ -768,6 +781,76 @@ struct GetReturnObjectManager {
CGF.EmitAutoVarInit(GroEmission);
Builder.CreateStore(Builder.getTrue(), GroActiveFlag);
}
+ // The GRO returns either when it is first suspended or when it completes
+ // without ever being suspended. The EmitGro
https://github.com/schittir converted_to_draft
https://github.com/llvm/llvm-project/pull/172179
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mpark wrote:
> > The patch itself looks good. But I like to test it internally before
> > merging. Hope I can make it in next week.
>
>
>
> In our internal workloads, I didn't see noticeable changes in End-to-End
> build process. Maybe the reason is we've already done a so called "bottom up"
zwuis wrote:
The issue was fixed. We appreciate the work you put in the PR though, thanks :)
https://github.com/llvm/llvm-project/pull/167046
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
https://github.com/zwuis closed https://github.com/llvm/llvm-project/pull/167046
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ChuanqiXu9 wrote:
> The patch itself looks good. But I like to test it internally before merging.
> Hope I can make it in next week.
In our internal workloads, I didn't see noticeable changes in End-to-End
compilations. But the good news is all things compile good. So LGTM.
>
> And also I th
@@ -4432,14 +4432,35 @@ class ASTDeclContextNameLookupTrait
TULocalDeclsMap.insert({D->getDeclName(), DeclIDsTy{ID}});
else
Iter->second.push_back(ID);
-continue;
+return;
}
case LookupVisibility::GenerallyVisibile:
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: None (sstwcw)
Changes
How the program figured out which lines to move along the aligned lines in
75c85bafb830e5. Aligning the lines caused the information to be out of date
for aligning different categories later on. It caused a
https://github.com/sstwcw created
https://github.com/llvm/llvm-project/pull/172242
How the program figured out which lines to move along the aligned lines in
75c85bafb830e5. Aligning the lines caused the information to be out of date
for aligning different categories later on. It caused a re
https://github.com/localspook updated
https://github.com/llvm/llvm-project/pull/172196
>From b284d180abdb752bd1a4fc2a1160ee1e9a73ea5c Mon Sep 17 00:00:00 2001
From: Victor Chernyakin
Date: Sat, 13 Dec 2025 21:46:57 -0800
Subject: [PATCH] [clang-tidy] Fix misplaced fix-its in
`modernize-use-ove
@@ -768,6 +779,41 @@ struct GetReturnObjectManager {
CGF.EmitAutoVarInit(GroEmission);
Builder.CreateStore(Builder.getTrue(), GroActiveFlag);
}
+
+ void EmitGroConv() {
+// GRO conversion is unnecessary when get_return_object's type matches the
+// coroutine
@@ -973,8 +1028,27 @@ void CodeGenFunction::EmitCoroutineBody(const
CoroutineBodyStmt &S) {
// shouldn't change the AST.
if (PreviousRetValue)
cast(Ret)->setRetValue(PreviousRetValue);
- }
+if (!GroManager.DirectEmit) {
+ // Send GRO conversion to Con
@@ -973,8 +1028,27 @@ void CodeGenFunction::EmitCoroutineBody(const
CoroutineBodyStmt &S) {
// shouldn't change the AST.
if (PreviousRetValue)
cast(Ret)->setRetValue(PreviousRetValue);
- }
+if (!GroManager.DirectEmit) {
+ // Send GRO conversion to Con
@@ -973,8 +1028,27 @@ void CodeGenFunction::EmitCoroutineBody(const
CoroutineBodyStmt &S) {
// shouldn't change the AST.
if (PreviousRetValue)
cast(Ret)->setRetValue(PreviousRetValue);
- }
+if (!GroManager.DirectEmit) {
+ // Send GRO conversion to Con
@@ -768,6 +779,41 @@ struct GetReturnObjectManager {
CGF.EmitAutoVarInit(GroEmission);
Builder.CreateStore(Builder.getTrue(), GroActiveFlag);
}
+
+ void EmitGroConv() {
+// GRO conversion is unnecessary when get_return_object's type matches the
+// coroutine
https://github.com/NewSigma updated
https://github.com/llvm/llvm-project/pull/151067
>From 01176d70b52235cc3ceb00bc33a285bcbd11bc58 Mon Sep 17 00:00:00 2001
From: NewSigma
Date: Sat, 13 Dec 2025 20:57:43 +0800
Subject: [PATCH] Resolve review comments
---
clang/lib/CodeGen/CGCoroutine.cpp
ChuanqiXu9 wrote:
> I believe that LLVM can be compiled >10x faster with my software.
If it is true, it implies the current build process spend > 90% time on build
systems and build system communications, it is really hard to believe...
> I can point potential HMake users to your comment and t
https://github.com/tclin914 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/172130
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zeyi2 requested changes to this pull request.
Sorry that I didn't have the condition to compile the code locally, so you may
need to verify this on your side.
---
I think removing the `hasInitStorage()` check will cause the existing Fix-it
logic to generate broken code. IMO
https://github.com/zeyi2 edited https://github.com/llvm/llvm-project/pull/172123
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zeyi2 edited https://github.com/llvm/llvm-project/pull/172123
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -289,8 +291,56 @@ def _clean_clang_tidy_output(self, output: str) -> str:
return ""
+class Doc8LintHelper(LintHelper):
+name: Final = "doc8"
+friendly_name: Final = "RST documentation linter"
-ALL_LINTERS = (ClangTidyLintHelper(),)
+def instructions(s
@@ -289,8 +291,56 @@ def _clean_clang_tidy_output(self, output: str) -> str:
return ""
+class Doc8LintHelper(LintHelper):
+name: Final = "doc8"
+friendly_name: Final = "RST documentation linter"
-ALL_LINTERS = (ClangTidyLintHelper(),)
+def instructions(s
HendrikHuebner wrote:
> Seems like you have been improving atomics support upstream, thanks for
> working on that. Given that one of the other PRs is basing off an older
> version of the incubator, please make sure this one is also up-to-date. I'd
> rather see this PR split in two (1) just add
https://github.com/HendrikHuebner updated
https://github.com/llvm/llvm-project/pull/168347
From 661817d4914aa54bf2d90748eb2aa448b1d967de Mon Sep 17 00:00:00 2001
From: hhuebner
Date: Mon, 17 Nov 2025 11:42:01 +0100
Subject: [PATCH 1/3] [CIR] Upstream __sync_up_and_fetch builtins
---
clang/lib
@@ -289,8 +291,56 @@ def _clean_clang_tidy_output(self, output: str) -> str:
return ""
+class Doc8LintHelper(LintHelper):
+name: Final = "doc8"
+friendly_name: Final = "RST documentation linter"
-ALL_LINTERS = (ClangTidyLintHelper(),)
+def instructions(s
https://github.com/zeyi2 edited https://github.com/llvm/llvm-project/pull/172220
___
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -289,8 +291,56 @@ def _clean_clang_tidy_output(self, output: str) -> str:
return ""
+class Doc8LintHelper(LintHelper):
+name: Final = "doc8"
+friendly_name: Final = "RST documentation linter"
-ALL_LINTERS = (ClangTidyLintHelper(),)
+def instructions(s
1 - 100 of 303 matches
Mail list logo
