llvmbot wrote:
@llvm/pr-subscribers-debuginfo
@llvm/pr-subscribers-clang-codegen
Author: Mariya Podchishchaeva (Fznamznon)
Changes
Whereas it is UB in terms of the standard to delete an array of objects via
pointer whose static type doesn't match its dynamic type, MSVC supports an
extens
https://github.com/Fznamznon updated
https://github.com/llvm/llvm-project/pull/126240
>From 04636bea1b873805af02dea865637d7125cee1e5 Mon Sep 17 00:00:00 2001
From: "Podchishchaeva, Mariya"
Date: Thu, 12 Dec 2024 08:57:37 -0800
Subject: [PATCH 1/2] [MS][clang] Add support for vector deleting des
https://github.com/devnexen updated
https://github.com/llvm/llvm-project/pull/125389
>From e40672e8137c0546b1604795901ccea8b15f7932 Mon Sep 17 00:00:00 2001
From: David Carlier
Date: Sun, 2 Feb 2025 09:36:50 +
Subject: [PATCH 1/3] [compiler-rt][rtsan] porting the sanitizer to FreeBSD.
Most
https://github.com/jhuber6 approved this pull request.
Makes sense, should be fine to transition since we just hit a release milestone.
https://github.com/llvm/llvm-project/pull/122336
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
Author: Timm Baeder
Date: 2025-02-07T14:24:15+01:00
New Revision: a69975d8349c1e0dcb86814df67d7160395b468e
URL:
https://github.com/llvm/llvm-project/commit/a69975d8349c1e0dcb86814df67d7160395b468e
DIFF:
https://github.com/llvm/llvm-project/commit/a69975d8349c1e0dcb86814df67d7160395b468e.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/126223
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nataraj-hates-MS-for-stealing-github wrote:
Reported `%p` problem in https://github.com/llvm/llvm-project/issues/126242
issue. In order it will not be lost and forgotten...
https://github.com/llvm/llvm-project/pull/123963
___
cfe-commits mailing list
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/126186
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Martin Storsjö
Date: 2025-02-07T15:30:29+02:00
New Revision: 2a6cfbd98516d902267a39e26ecbbff8c8121b01
URL:
https://github.com/llvm/llvm-project/commit/2a6cfbd98516d902267a39e26ecbbff8c8121b01
DIFF:
https://github.com/llvm/llvm-project/commit/2a6cfbd98516d902267a39e26ecbbff8c8121b01.diff
https://github.com/mstorsjo closed
https://github.com/llvm/llvm-project/pull/126139
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 created
https://github.com/llvm/llvm-project/pull/126248
Summary:
This patch handles the use of `-Xarch_ -Wl,foo` to send an
argument to the linker for the embedded offloading jobs in the linker
wrapper. This makes it equivalent to `-Xoffload-linker foo`.
>From 10727
jhuber6 wrote:
> We already have a host-side `libomptarget`:
>
> https://github.com/llvm/llvm-project/blob/cfd8980fe79872feb9ed81959aa018681c7b3354/offload/src/CMakeLists.txt#L9
We can have conflicting names since they're in separate install directories.
I'd like this to have a more 'standard'
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Joseph Huber (jhuber6)
Changes
Summary:
This patch handles the use of `-Xarch_ -Wl,foo` to send an
argument to the linker for the embedded offloading jobs in the linker
wrapper. This makes it equivalent to `-Xoffload-linker foo
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Joseph Huber (jhuber6)
Changes
Summary:
This patch handles the use of `-Xarch_ -Wl,foo` to send an
argument to the linker for the embedded offloading jobs in the linker
wrapper. This makes it equivalent to `-Xoffload-linker foo`.
--
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/126248
>From 10727e968d69bc3f0848a1cbbd0f3e5012b09d5f Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Fri, 7 Feb 2025 08:31:08 -0600
Subject: [PATCH 1/2] [Clang] Forward `-Xarch_ -Wl,foo` for GPU
toolchains
Summary
https://github.com/to268 updated https://github.com/llvm/llvm-project/pull/79845
>From 7132c5d853fafb3e956c89b190313bf25ef1c773 Mon Sep 17 00:00:00 2001
From: Guillot Tony
Date: Thu, 30 Jan 2025 01:12:13 +0100
Subject: [PATCH] N3006 base
---
clang/docs/ReleaseNotes.rst | 2 +
@@ -912,3 +924,33 @@ void top_malloc_no_crash_fn() {
free(pptr);
}
} // namespace alloca_region_pointer
+
+namespace true_negatives_return_expressions {
+struct Container { int *x; };
+
+int test2() {
+ int x = 14;
Xazax-hun wrote:
Could you add an example
https://github.com/Fznamznon created
https://github.com/llvm/llvm-project/pull/126240
Whereas it is UB in terms of the standard to delete an array of objects via
pointer whose static type doesn't match its dynamic type, MSVC supports an
extension allowing to do it.
Aside from array deletion no
https://github.com/ampandey-1995 updated
https://github.com/llvm/llvm-project/pull/126186
>From 43433affa692b2690bc3dbb738ab5c76811e68e8 Mon Sep 17 00:00:00 2001
From: Amit Pandey
Date: Fri, 7 Feb 2025 12:11:23 +0530
Subject: [PATCH] [Driver][ROCm][OpenMP] Fix default ockl linking for OpenMP.
jhuber6 wrote:
> I'm not sure if calling it `libomp.a` is a good idea. I personally find it
> confusing because "libomp" is already an established name—it refers to the
> OpenMP host runtime contributed by Intel, just as "libgomp" is the OpenMP
> host runtime for GCC. These names are well-reco
https://github.com/pfusik updated
https://github.com/llvm/llvm-project/pull/126245
>From 666c1d15556579707c1ebce07ed2774fe13437ec Mon Sep 17 00:00:00 2001
From: Piotr Fusik
Date: Fri, 7 Feb 2025 14:58:34 +0100
Subject: [PATCH 1/2] [BoundsSafety][doc] Add a missing newline in C snippet
---
cla
https://github.com/pfusik created
https://github.com/llvm/llvm-project/pull/126247
None
>From 7aa0008aafc776941151dec16f52725b8958e730 Mon Sep 17 00:00:00 2001
From: Piotr Fusik
Date: Fri, 7 Feb 2025 15:19:11 +0100
Subject: [PATCH] [BoundsSafety][doc] Fix a typo
---
clang/docs/BoundsSafety.r
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Piotr Fusik (pfusik)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/126247.diff
2 Files Affected:
- (modified) clang/docs/BoundsSafety.rst (+3-3)
- (modified) clang/docs/BoundsSafetyImplPlans.rst (+1-1)
``
Meinersbur wrote:
We already have a host-side `libomptarget`:
https://github.com/llvm/llvm-project/blob/cfd8980fe79872feb9ed81959aa018681c7b3354/offload/src/CMakeLists.txt#L9
https://github.com/llvm/llvm-project/pull/126143
___
cfe-commits mailing list
@@ -43,6 +44,21 @@ static __rtsan::Context &GetContextForThisThreadImpl() {
return *current_thread_context;
}
+#else
+
+// On FreeBSD, pthread api cannot be used as calloc is called under the hood
+// at library initialization time.
devnexen wrote:
So I kno
https://github.com/devnexen edited
https://github.com/llvm/llvm-project/pull/125389
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: None (chrisPyr)
Changes
#125983
---
Patch is 216.27 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/126243.diff
128 Files Affected:
- (modified) bolt/lib/Core/BinaryContext.cpp (+4-
llvmbot wrote:
@llvm/pr-subscribers-bolt
@llvm/pr-subscribers-clang
Author: None (chrisPyr)
Changes
#125983
---
Patch is 216.27 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/126243.diff
128 Files Affected:
- (modified) bolt/lib/Core/Binar
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/126143
>From cda46da874a083ea89bd9bb31616806b0dab7692 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Thu, 6 Feb 2025 15:54:19 -0600
Subject: [PATCH 1/2] [OpenMP] Remove 'libomptarget.devicertl.a' fatbinary and
use
https://github.com/yxsamliu approved this pull request.
https://github.com/llvm/llvm-project/pull/126186
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/pfusik created
https://github.com/llvm/llvm-project/pull/126245
None
>From 666c1d15556579707c1ebce07ed2774fe13437ec Mon Sep 17 00:00:00 2001
From: Piotr Fusik
Date: Fri, 7 Feb 2025 14:58:34 +0100
Subject: [PATCH] [BoundsSafety][doc] Add a missing newline in C snippet
---
c
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Piotr Fusik (pfusik)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/126245.diff
1 Files Affected:
- (modified) clang/docs/BoundsSafety.rst (+3-2)
``diff
diff --git a/clang/docs/BoundsSafety.rst b/clang/docs
https://github.com/pfusik edited
https://github.com/llvm/llvm-project/pull/126245
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
w2yehia wrote:
Thank you to the reviewers.
I'll merge this tonight if I get no further feedback.
https://github.com/llvm/llvm-project/pull/124353
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
broxigarchen wrote:
> Same suggestion as in [#119750
> (comment)](https://github.com/llvm/llvm-project/pull/119750#discussion_r1941297212).
I see. I think this is better to be done seperately
https://github.com/llvm/llvm-project/pull/125706
___
cfe-c
https://github.com/broxigarchen closed
https://github.com/llvm/llvm-project/pull/125706
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9203,7 +9203,9 @@ void LinkerWrapper::ConstructJob(Compilation &C, const
JobAction &JA,
ArgStringList CompilerArgs;
ArgStringList LinkerArgs;
for (Arg *A : C.getArgsForToolChain(TC, /*BoundArch=*/"", Kind)) {
-if (ShouldForward(CompilerOptions, A)
jurahul wrote:
I understand this is mostly mechanical changes, but wondering if review wise it
will help if its split up into 4-5 PRs. For example, bolt, clang, flag, llvm,
mlir etc.
https://github.com/llvm/llvm-project/pull/126243
___
cfe-commits ma
vbvictor wrote:
Ping
https://github.com/llvm/llvm-project/pull/121291
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1146,11 +1146,29 @@ bool
ExtractAPIVisitorBase::VisitTypedefNameDecl(
StringRef Name = Decl->getName();
+ auto nameMatches = [&Name](TagDecl *TagDecl) {
+StringRef TagName = TagDecl->getName();
+
+if (TagName == Name)
+ return true;
+
+// Also check w
saiislam wrote:
> > We already have a host-side `libomptarget`:
> > https://github.com/llvm/llvm-project/blob/cfd8980fe79872feb9ed81959aa018681c7b3354/offload/src/CMakeLists.txt#L9
>
> We can have conflicting names since they're in separate install directories.
> I'd like this to have a more 's
jurahul wrote:
Yeah, that’s why the description said most but not all. We should get link
failures if one of these is made static.
On Fri, Feb 7, 2025 at 7:17 AM Joseph Huber ***@***.***>
wrote:
> ***@***. commented on this pull request.
>
> This should definitely be split up. Also some opt
jhuber6 wrote:
> So I should do like open 5 branches and make a PR for each?
>
> As for the cases mentioned, when I tried compile all projects (compile with
> -DLLVM_ENABLE_PROJECTS=all), I did encounter such cases when compiling and
> I've fixed them all. Now all projects can be successfully
@@ -9203,7 +9203,9 @@ void LinkerWrapper::ConstructJob(Compilation &C, const
JobAction &JA,
ArgStringList CompilerArgs;
ArgStringList LinkerArgs;
for (Arg *A : C.getArgsForToolChain(TC, /*BoundArch=*/"", Kind)) {
-if (ShouldForward(CompilerOptions, A)
@@ -438,18 +445,28 @@ void llvm::PointerMayBeCaptured(const Value *V,
CaptureTracker *Tracker,
};
while (!Worklist.empty()) {
const Use *U = Worklist.pop_back_val();
-switch (DetermineUseCaptureKind(*U, IsDereferenceableOrNull)) {
-case UseCaptureKind::NO_CAPTU
https://github.com/nikic approved this pull request.
LGTM
> I don't know if directly using the CodeGenModule here is the best solution.
> Let me know if there's something better to use.
I think that's ok. The alternative would be Builder.getContext().
https://github.com/llvm/llvm-project/pull
nikic wrote:
Well, I guess you could also use Builder.getPtrTy() for the whole thing...
https://github.com/llvm/llvm-project/pull/124782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex approved this pull request.
https://github.com/llvm/llvm-project/pull/126248
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9203,7 +9203,9 @@ void LinkerWrapper::ConstructJob(Compilation &C, const
JobAction &JA,
ArgStringList CompilerArgs;
ArgStringList LinkerArgs;
for (Arg *A : C.getArgsForToolChain(TC, /*BoundArch=*/"", Kind)) {
-if (ShouldForward(CompilerOptions, A)
vbvictor wrote:
Ping
https://github.com/llvm/llvm-project/pull/122951
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
chrisPyr wrote:
So I should do like open 5 branches and make a PR for each?
As for the cases mentioned, when I tried compile all projects (compile with
-DLLVM_ENABLE_PROJECTS=all), I did encounter such cases when compiling and I've
fixed them all.
Now all projects can be successfully built, s
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/126143
>From 4d325d66f9792235f4cdcbcc8b032a5498dcb407 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Thu, 6 Feb 2025 15:54:19 -0600
Subject: [PATCH] [OpenMP] Remove 'libomptarget.devicertl.a' fatbinary and use
stat
chrisPyr wrote:
OK, I'll do it.
Except for checking it on CI, is there any other method I can test locally?
E.g. what options should I add
I've tried compile with all project enabled, I thought by that way all changes
were already being tested.
However, the CI failed, so it seems what I thought
https://github.com/gutron created
https://github.com/llvm/llvm-project/pull/126254
vc-git-program is referenced but not defined -- vc-git provides the definition.
>From 598b1c0df4991b1fb63b54dd358b40103912b80b Mon Sep 17 00:00:00 2001
From: Ron Gut <65685404+gut...@users.noreply.github.com>
Dat
https://github.com/Xazax-hun approved this pull request.
https://github.com/llvm/llvm-project/pull/126094
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/126243
___
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-format
Author: Ron Gut (gutron)
Changes
vc-git-program is referenced but not defined -- vc-git provides the definition.
---
Full diff: https://github.com/llvm/llvm-project/pull/126254.diff
1 Files Affected:
- (modified) clang/tools/clang-format
@@ -438,18 +445,28 @@ void llvm::PointerMayBeCaptured(const Value *V,
CaptureTracker *Tracker,
};
while (!Worklist.empty()) {
const Use *U = Worklist.pop_back_val();
-switch (DetermineUseCaptureKind(*U, IsDereferenceableOrNull)) {
-case UseCaptureKind::NO_CAPTU
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/126142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/126142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/spall edited https://github.com/llvm/llvm-project/pull/126258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/spall edited https://github.com/llvm/llvm-project/pull/126258
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/126142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/126142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/126142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/126142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 edited
https://github.com/llvm/llvm-project/pull/126243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 commented:
This should definitely be split up. Also some options are referenced in
multiple places, i.e.
```
// foo.h
extern cl::opt <...>
// foo.cpp
cl::opt <...>
use opt
//bar.cpp
use opt.
```
https://github.com/llvm/llvm-project/pull/126243
__
jhuber6 wrote:
> Should it be named as `liboffload` or `liboffload.devicertl`?
I don't want `offload` since I think that should honestly be a separate
library. I'll just call it `ompdevice` and be done with it.
https://github.com/llvm/llvm-project/pull/126143
__
https://github.com/junlarsen updated
https://github.com/llvm/llvm-project/pull/124771
>From f3bbd6f0db4109016d98fe15a2b086b491df2d93 Mon Sep 17 00:00:00 2001
From: Mats Jun Larsen
Date: Wed, 29 Jan 2025 00:31:32 +0900
Subject: [PATCH 1/2] [CodeGen] Replace of PointerType::get(Type) with opaque
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/126143
>From e3962f9bdab7de8682712857d7b284af2ac99fc3 Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Thu, 6 Feb 2025 15:54:19 -0600
Subject: [PATCH] [OpenMP] Remove 'libomptarget.devicertl.a' fatbinary and use
stat
jhuber6 wrote:
I figured that calling it `libomptarget` was fine because we already have
`libomptarget-amdgpu.bc` so what's the big difference in
`amdgcn-amd-amdhsa/libomptarget.a`. But I can change it.
https://github.com/llvm/llvm-project/pull/126143
__
kasuga-fj wrote:
Thanks for the comments. I'll try to work on it in the near future.
https://github.com/llvm/llvm-project/pull/125830
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -438,18 +445,28 @@ void llvm::PointerMayBeCaptured(const Value *V,
CaptureTracker *Tracker,
};
while (!Worklist.empty()) {
const Use *U = Worklist.pop_back_val();
-switch (DetermineUseCaptureKind(*U, IsDereferenceableOrNull)) {
-case UseCaptureKind::NO_CAPTU
junlarsen wrote:
I think I'll do that instead. Sounds a lot more reasonable in my opinion
https://github.com/llvm/llvm-project/pull/124782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/120896
>From 295df258043ef5a87ae603eedd308b863bad7b59 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sun, 22 Dec 2024 15:14:30 +0200
Subject: [PATCH 1/7] [Clang] allow restrict qualifier for array types with
poin
https://github.com/junlarsen updated
https://github.com/llvm/llvm-project/pull/124771
>From f3bbd6f0db4109016d98fe15a2b086b491df2d93 Mon Sep 17 00:00:00 2001
From: Mats Jun Larsen
Date: Wed, 29 Jan 2025 00:31:32 +0900
Subject: [PATCH 1/2] [CodeGen] Replace of PointerType::get(Type) with opaque
jurahul wrote:
You can check the CI logs for the exact command line it uses and replicate
it locally.
On Fri, Feb 7, 2025 at 7:46 AM chrisPyr ***@***.***> wrote:
> OK, I'll do it.
> Except for checking it on CI, is there any other method I can test
> locally? E.g. what options should I add
> I'
jurahul wrote:
@arsenm did you intent to approve it without splitting? Is this trivial enough
to not need splitting as long as CI checks pass?
https://github.com/llvm/llvm-project/pull/126243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
gutron wrote:
@goldsteinn -- this is a follow-up to your commit 32be90d
https://github.com/llvm/llvm-project/pull/126254
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
I don't care about splitting or not; either way
https://github.com/llvm/llvm-project/pull/126243
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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/AaronBallman closed
https://github.com/llvm/llvm-project/pull/79845
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
@to268 and I spoke about this offline and we decided to close this PR because
it's heading in the wrong direction. We discussed a different direction that
focuses solely on implementing the logic to handle `%select{declaration using a
deduced type|constexpr declaration}0 %1
https://github.com/goldsteinn approved this pull request.
LGTM and Thank you. I'm going to slightly rename the commit message before
merging to refer to the correct component.
https://github.com/llvm/llvm-project/pull/126254
___
cfe-commits mailing li
https://github.com/goldsteinn closed
https://github.com/llvm/llvm-project/pull/126254
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ron Gut
Date: 2025-02-07T11:04:24-05:00
New Revision: d0170424b7250bf5cda0e6253ec62112a6e1c92a
URL:
https://github.com/llvm/llvm-project/commit/d0170424b7250bf5cda0e6253ec62112a6e1c92a
DIFF:
https://github.com/llvm/llvm-project/commit/d0170424b7250bf5cda0e6253ec62112a6e1c92a.diff
LOG:
github-actions[bot] wrote:
@gutron 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 build,
https://github.com/jmorse commented:
This looks great, thanks for putting this all together, one nit inline,
Is this intended to be cherry-picked into the release branch or not? Seeing how
the original motivation was to ease downstream compatibilities, I'd assumed
there'd be the need for somet
MacDue wrote:
Given it looks like #123787 won't be accepted (due to compile time
regressions), I think we should restrict this lowering to AArch64 (and other
supported targets) until there's a solution for targets without AA in codegen
too. Does that seem reasonable?
https://github.com/llvm
https://github.com/jmorse edited
https://github.com/llvm/llvm-project/pull/126059
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
> As for the cases mentioned, when I tried compile all projects (compile with
> -DLLVM_ENABLE_PROJECTS=all), I did encounter such cases when compiling and
> I've fixed them all.
> Now all projects can be successfully built, so I think it should be fine?
Just to explain why
@@ -1708,19 +1706,9 @@ static void insertDbgValueOrDbgVariableRecord(DIBuilder
&Builder, Value *DV,
static void insertDbgValueOrDbgVariableRecordAfter(
DIBuilder &Builder, Value *DV, DILocalVariable *DIVar, DIExpression
*DIExpr,
-const DebugLoc &NewLoc, BasicBlock::i
DavidSpickett wrote:
> You can check the CI logs for the exact command line it uses and replicate it
> locally.
Should be towards the start of the log, search for `cmake ` to find the command.
https://github.com/llvm/llvm-project/pull/126243
___
cfe-
https://github.com/junlarsen updated
https://github.com/llvm/llvm-project/pull/124782
>From bc77542fe58bdc94566a9a589039856de50617b0 Mon Sep 17 00:00:00 2001
From: Mats Jun Larsen
Date: Wed, 29 Jan 2025 01:29:01 +0900
Subject: [PATCH 1/2] [Clang][TableGen] Use PointerType::get(Context) in MVE
https://github.com/junlarsen updated
https://github.com/llvm/llvm-project/pull/124782
>From bc77542fe58bdc94566a9a589039856de50617b0 Mon Sep 17 00:00:00 2001
From: Mats Jun Larsen
Date: Wed, 29 Jan 2025 01:29:01 +0900
Subject: [PATCH 1/3] [Clang][TableGen] Use PointerType::get(Context) in MVE
https://github.com/junlarsen updated
https://github.com/llvm/llvm-project/pull/124771
>From 13d65a0cc95fe53f959db343e5be07773ee53f28 Mon Sep 17 00:00:00 2001
From: Mats Jun Larsen
Date: Wed, 29 Jan 2025 00:31:32 +0900
Subject: [PATCH 1/2] [CodeGen] Replace of PointerType::get(Type) with opaque
https://github.com/nikic updated
https://github.com/llvm/llvm-project/pull/125880
>From 2f698e27ae61b91019544cc707c134e0aec9ecd3 Mon Sep 17 00:00:00 2001
From: Nikita Popov
Date: Thu, 30 Jan 2025 12:08:01 +0100
Subject: [PATCH 1/5] [CaptureTracking][FunctionAttrs] Add support for
CaptureInfo
@@ -210,7 +210,7 @@ class PointerType : public Type {
return Name + " *";
}
std::string llvmName() const override {
-return "llvm::PointerType::getUnqual(" + Pointee->llvmName() + ")";
+return "llvm::PointerType::getUnqual(Builder.getPtrTy())";
401 - 500 of 527 matches
Mail list logo
