https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83237
>From f2e53e44eebab4720a1dbade24fcb14d698fb03f Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH 1/4] [Serialization] Code cleanups and polish 83233
---
clang/in
ChuanqiXu9 wrote:
> Here is the reproducer. Like before, builds at head and fails with the patch
> applied. After unpacking the archive just run
>
> ```
> $ CLANGPP= ./build.sh
> ```
>
> Note the comment in `build.sh` about the system libc++ required for it
> (libc++ 17 worked, 15 and 16 did
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83237
>From f2e53e44eebab4720a1dbade24fcb14d698fb03f Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH 1/6] [Serialization] Code cleanups and polish 83233
---
clang/in
ChuanqiXu9 wrote:
I think now I understand the problem. The root cause happens in
https://github.com/llvm/llvm-project/blob/175aa864f33786f3a6a4ee7381cbcafd0758501a/clang/lib/Serialization/MultiOnDiskHashTable.h#L329
The description in () is optional. You can skip it if you're not interested it
@@ -1827,6 +1833,12 @@ void ASTDeclWriter::VisitVarTemplateDecl(VarTemplateDecl
*D) {
void ASTDeclWriter::VisitVarTemplateSpecializationDecl(
VarTemplateSpecializationDecl *D) {
+ // FIXME: We need to load the "logical" first declaration before writing
+ // the Redeclar
https://github.com/ChuanqiXu9 commented:
The patch looks good to me except the thing I mentioned in
https://github.com/llvm/llvm-project/pull/99282#pullrequestreview-2265588601
https://github.com/llvm/llvm-project/pull/99283
___
llvm-branch-commits ma
https://github.com/ChuanqiXu9 edited
https://github.com/llvm/llvm-project/pull/99283
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1455,6 +1462,74 @@ struct SwitchCoroutineSplitter {
setCoroInfo(F, Shape, Clones);
}
+ // Create a variant of ramp function that does not perform heap allocation
+ // for a switch ABI coroutine.
+ //
+ // The newly split `.noalloc` ramp function has the following
@@ -1455,6 +1462,74 @@ struct SwitchCoroutineSplitter {
setCoroInfo(F, Shape, Clones);
}
+ // Create a variant of ramp function that does not perform heap allocation
+ // for a switch ABI coroutine.
+ //
+ // The newly split `.noalloc` ramp function has the following
@@ -26,6 +26,10 @@ bool declaresIntrinsics(const Module &M,
const std::initializer_list);
void replaceCoroFree(CoroIdInst *CoroId, bool Elide);
+void suppressCoroAllocs(CoroIdInst *CoroId);
ChuanqiXu9 wrote:
Let's add some comments for
@@ -2049,6 +2055,21 @@ the coroutine must reach the final suspend point when it
get destroyed.
This attribute only works for switched-resume coroutines now.
+coro_elide_safe
+---
+
+When a Call or Invoke instruction is marked with `coro_elide_safe`,
+CoroAnnotati
@@ -0,0 +1,147 @@
+//===- CoroAnnotationElide.cpp - Elide attributed safe coroutine calls
===//
+//
+// 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
@@ -0,0 +1,147 @@
+//===- CoroAnnotationElide.cpp - Elide attributed safe coroutine calls
===//
+//
+// 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
@@ -0,0 +1,147 @@
+//===- CoroAnnotationElide.cpp - Elide attributed safe coroutine calls
===//
+//
+// 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
@@ -0,0 +1,147 @@
+//===- CoroAnnotationElide.cpp - Elide attributed safe coroutine calls
===//
+//
+// 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/ChuanqiXu9 edited
https://github.com/llvm/llvm-project/pull/99285
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -0,0 +1,147 @@
+//===- CoroAnnotationElide.cpp - Elide attributed safe coroutine calls
===//
+//
+// 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/ChuanqiXu9 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/99283
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/ChuanqiXu9 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/99285
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
Very sorry that I failed to count
https://github.com/llvm/llvm-project/issues/102933 as a regression support
initially. I thought it was other problems. My bad. And back to the issue
itself, it shows that the fix is pretty simple and it is majorly an oversight
in the develop
ChuanqiXu9 wrote:
> > what the code does is: when we write a on-disk hash table, try to write the
> > imported merged hash table in the same process so that we don't need to
> > read these tables again. However, in line 329 the function will try to omit
> > the data from imported table with th
ChuanqiXu9 wrote:
> > what the code does is: when we write a on-disk hash table, try to write the
> > imported merged hash table in the same process so that we don't need to
> > read these tables again. However, in line 329 the function will try to omit
> > the data from imported table with th
Author: Chuanqi Xu
Date: 2021-01-20T18:59:00+08:00
New Revision: c1bc7981babcae20247650a4b8adab9c0c97890a
URL:
https://github.com/llvm/llvm-project/commit/c1bc7981babcae20247650a4b8adab9c0c97890a
DIFF:
https://github.com/llvm/llvm-project/commit/c1bc7981babcae20247650a4b8adab9c0c97890a.diff
LO
Author: Chuanqi Xu
Date: 2020-12-15T20:50:46+08:00
New Revision: 8b48d24373204fc2fe6aac1f1f850fa3b6c18445
URL:
https://github.com/llvm/llvm-project/commit/8b48d24373204fc2fe6aac1f1f850fa3b6c18445
DIFF:
https://github.com/llvm/llvm-project/commit/8b48d24373204fc2fe6aac1f1f850fa3b6c18445.diff
LO
ChuanqiXu9 wrote:
@ilya-biryukov gentle ping~
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
> We've hit only one correctness issue that we don't yet have a reproducer for,
> but it gives this error:
>
> ```shell
> Eigen/.../plugins/CommonCwiseBinaryOps.inc:47:29: error: inline function
> 'Eigen::operator*' is not defined [-Werror,-Wundefined-inline]
> ```
>
> I'll
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83237
>From 5df9f526236cff3b2212088bf6bf52c6802044e2 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH] [Serialization] Code cleanups and polish 83233
fmt
load special
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83233
>From b61bcaafa3700b0797772df58710158eb44eaa69 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH] [Serialization] Introduce OnDiskHashTable for specializations
Fo
ChuanqiXu9 wrote:
> There were some merge conflicts with main, I've managed to resolve them to
> this: https://github.com/ilya-biryukov/llvm-project/tree/pr83237_merged
>
> I am now trying to bootstrap the compiler with that version, but it would be
> useful if someone rebased this PR against
ChuanqiXu9 wrote:
> @usx95 may be able to help with the reproducer.
>
> In the meantime, I'm trying to collect some information on the compile times.
> So far it looks like we have a ~10-15x compile time regression on some
> translation units. Without this patch `-ftime-report` shows:
>
> ```
ChuanqiXu9 wrote:
I tried to take a look at eigen and it looks like the declaration looks well
and I had no clue how that happens. A reproducer may be necessary here to
proceed. Thanks in advance.
https://github.com/llvm/llvm-project/pull/83237
___
l
ChuanqiXu9 wrote:
> > I tried to take a look at eigen and it looks like the declaration looks
> > well and I had no clue how that happens. A reproducer may be necessary here
> > to proceed. Thanks in advance.
>
> I can reproduce using the following sources and invocations outlined in
> `run.s
ChuanqiXu9 wrote:
> > I tried to take a look at eigen and it looks like the declaration looks
> > well and I had no clue how that happens. A reproducer may be necessary here
> > to proceed. Thanks in advance.
>
> I can reproduce using the following sources and invocations outlined in
> `run.s
ChuanqiXu9 wrote:
> @ChuanqiXu9 is this one ready to be merged?
Yes, I believe this is ready to be merged.
https://github.com/llvm/llvm-project/pull/114197
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/
https://github.com/ChuanqiXu9 approved this pull request.
https://github.com/llvm/llvm-project/pull/109077
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
> @ChuanqiXu9 this looks safe enough to be picked. Does the PR look fine to you?
Yes, I'll try to approve it formally.
https://github.com/llvm/llvm-project/pull/109076
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.ll
https://github.com/ChuanqiXu9 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/109076
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
> If you can put up another patch I'll just wait with reverting.
trying : )
https://github.com/llvm/llvm-project/pull/109077
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83237
>From f2e53e44eebab4720a1dbade24fcb14d698fb03f Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH 1/7] [Serialization] Code cleanups and polish 83233
---
clang/in
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/109762
>From 4c51d827e58aaa8c5b3d75b3b61a43627ab53491 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Tue, 24 Sep 2024 15:37:02 +0800
Subject: [PATCH] [C++20] [Modules] Add Decl::isFromGlobalModule
---
clang/inclu
ChuanqiXu9 wrote:
@ilya-biryukov thanks for the high quality reproducer! I've reproduced it and
fixed it in the newest commit.
The root cause of the failure is the same with the last time: merging
specializations with the same key. The previous fix is not clear since, I just
found, the underl
ChuanqiXu9 wrote:
Oh, maybe some implicit conflicts. I'll add one manually
https://github.com/llvm/llvm-project/pull/109077
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bra
Author: Chuanqi Xu
Date: 2024-09-24T15:41:11+08:00
New Revision: 4c51d827e58aaa8c5b3d75b3b61a43627ab53491
URL:
https://github.com/llvm/llvm-project/commit/4c51d827e58aaa8c5b3d75b3b61a43627ab53491
DIFF:
https://github.com/llvm/llvm-project/commit/4c51d827e58aaa8c5b3d75b3b61a43627ab53491.diff
LO
ChuanqiXu9 wrote:
@alexfh thank you very much!
@vgvassilev but we have to provide similar mechanism, so it is allowed to get
all the specializations for a templated decl.
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing
ChuanqiXu9 wrote:
> do we store all template variable specializations in the same place in the
> map including the partial ones?
Yes, we identify if they are partial by an additional bit.
For the solution, given there might be other places need to load the
specializations, how about removing
ChuanqiXu9 wrote:
> > > do we store all template variable specializations in the same place in
> > > the map including the partial ones?
> >
> >
> > Yes, we identify if they are partial by an additional bit.
> > For the solution, given there might be other places need to load the
> > speciali
ChuanqiXu9 wrote:
@vgvassilev @ilya-biryukov @alexfh If I read correctly, the only blocker issue
is the above reported performance issue. And I tried to split partial
specialization from the full specialization table to avoid merging tables again
and again. Could you please take another round
ChuanqiXu9 wrote:
Sorry for the not good stacked PR operation.
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83237
>From 493cb7cdafeb22efa9e9f3c1954b6de1e2665365 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH 1/2] [Serialization] Code cleanups and polish 83233
---
clang/in
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83233
>From f565dd3f156bbdf608be6643208c40f02b4f0e83 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Wed, 28 Feb 2024 11:41:53 +0800
Subject: [PATCH] [Serialization] Introduce OnDiskHashTable for specializations
Fo
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/83233
>From 11726437efb760c9f2aba9b2258337b2b8eb4bb6 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Fri, 8 Nov 2024 17:19:33 +0800
Subject: [PATCH] [Serialization] Introduce OnDiskHashTable for specializations
---
ChuanqiXu9 wrote:
@alexfh gentle ping
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
> Once again, thanks for bearing with us and addressing all the issues. The
> latest version seems both correct and does not cause performance regressions.
> Let's land this!
>
> PS please note that the resolution of our compile-time profiling instruments
> is not that great
ChuanqiXu9 wrote:
Note that I'll merge this and https://github.com/llvm/llvm-project/pull/83233
and https://github.com/llvm/llvm-project/pull/83108 into a commit and commit it
directly to trunk. Since these prs are split initially to make it clear to be
reviewed. But during the review, we alwa
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/83233
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/ChuanqiXu9 milestoned
https://github.com/llvm/llvm-project/pull/121739
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
@jijjijj Please note the commits in the summary
https://github.com/llvm/llvm-project/pull/121739
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/ChuanqiXu9 approved this pull request.
The change itself looks good to me.
https://github.com/llvm/llvm-project/pull/121739
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
ChuanqiXu9 wrote:
Sent https://github.com/llvm/llvm-project/pull/119333
It looks like the lldb's failure is from we forgot to update the
ExternalASTConsumer (I met this the second time. I am wondering if we can make
it more automatically). The other windows failure is a pattern match failure.
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
Sent
https://github.com/llvm/llvm-project/commit/b5bd1928c6d43bc525a4e3fb65d2c750d61e
and see https://github.com/llvm/llvm-project/pull/83237#issuecomment-2521945547
https://github.com/llvm/llvm-project/pull/83233
___
llvm-branc
ChuanqiXu9 wrote:
Sent
https://github.com/llvm/llvm-project/commit/b5bd1928c6d43bc525a4e3fb65d2c750d61e
https://github.com/llvm/llvm-project/pull/83237
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/
ChuanqiXu9 wrote:
CC @jijjijj
https://github.com/llvm/llvm-project/pull/121739
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ChuanqiXu9 wrote:
> @ChuanqiXu9 What do you think about merging this PR to the release branch?
It is necessary for using std module on MacOS with CMake. I think it is
important.
https://github.com/llvm/llvm-project/pull/122844
___
llvm-branch-commits
ChuanqiXu9 wrote:
The tests are failing and it looks relevent.
https://github.com/llvm/llvm-project/pull/121046
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -67,35 +68,84 @@ Semantic Highlighting
Compile flags
^
+- Fixed a bug where clangd would unnecessarily reparse open files whose
+ compile command did not change when receiving a new compile command
+ via an LSP `workspace/configuration` request (#GH115438)
+
ChuanqiXu9 wrote:
This is a simple fix to a problem with a (relatively) long history. I think it
is good to backport this.
https://github.com/llvm/llvm-project/pull/133198
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https:/
ChuanqiXu9 wrote:
Fixed a crash for clangd with modules.
https://github.com/llvm/llvm-project/pull/143647
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/ChuanqiXu9 updated
https://github.com/llvm/llvm-project/pull/143647
>From eabe027feb43511d2d2709df5b88367a2cb622d5 Mon Sep 17 00:00:00 2001
From: fleeting-xx
Date: Thu, 5 Jun 2025 20:33:11 -0500
Subject: [PATCH 1/2] [clangd] [Modules] Fix to correctly handle module
dependenc
https://github.com/ChuanqiXu9 created
https://github.com/llvm/llvm-project/pull/143647
Backport https://github.com/llvm/llvm-project/pull/142828 manually
This is helpful to avoid crashes in clangd.
>From 31fab1b1c7564955f94fb64aa0603125c8366a11 Mon Sep 17 00:00:00 2001
From: fleeting-xx
Date:
@@ -1213,11 +1213,17 @@ static void insertSpills(const FrameDataInfo
&FrameData, coro::Shape &Shape) {
for (const auto &A : FrameData.Allocas) {
AllocaInst *Alloca = A.Alloca;
UsersToUpdate.clear();
-for (User *U : Alloca->users()) {
+for (User *U : make_earl
101 - 171 of 171 matches
Mail list logo
