https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/111958
___
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/aaupov updated
https://github.com/llvm/llvm-project/pull/112061
___
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/aaupov updated
https://github.com/llvm/llvm-project/pull/112061
___
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/maksfb approved this pull request.
LGTM. I'd mark this PR as "NFCI".
https://github.com/llvm/llvm-project/pull/112061
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/111954
___
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/var-const approved this pull request.
https://github.com/llvm/llvm-project/pull/110677
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@llvm/pr-subscribers-bolt
Author: Amir Ayupov (aaupov)
Changes
For a large binary with BAT section of size 38 MB with ~170k maps,
reduces writeMaps time from 70s down to 1s.
The inefficiency was in the use of std::distance with std::map::iterator
which doesn't provide rando
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/112061
For a large binary with BAT section of size 38 MB with ~170k maps,
reduces writeMaps time from 70s down to 1s.
The inefficiency was in the use of std::distance with std::map::iterator
which doesn't provide random
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
perry-ca wrote:
I'm looking into that question. The first thing I noticed was the temp file is
being created and read as a binary file. I also noticed that the output of the
script is not being redirected into the te
llvmbot wrote:
@llvm/pr-subscribers-lld-elf
Author: None (llvmbot)
Changes
Backport 77aa8257acbd773c0c430cd962da1bcfbd5ee94b
Requested by: @androm3da
---
Full diff: https://github.com/llvm/llvm-project/pull/112040.diff
2 Files Affected:
- (modified) lld/ELF/Arch/Hexagon.cpp (+17-3)
-
llvmbot wrote:
@llvm/pr-subscribers-backend-hexagon
Author: None (llvmbot)
Changes
Backport 77aa8257acbd773c0c430cd962da1bcfbd5ee94b
Requested by: @androm3da
---
Full diff: https://github.com/llvm/llvm-project/pull/112040.diff
2 Files Affected:
- (modified) lld/ELF/Arch/Hexagon.cpp (+
llvmbot wrote:
@llvm/pr-subscribers-lld
Author: None (llvmbot)
Changes
Backport 77aa8257acbd773c0c430cd962da1bcfbd5ee94b
Requested by: @androm3da
---
Full diff: https://github.com/llvm/llvm-project/pull/112040.diff
2 Files Affected:
- (modified) lld/ELF/Arch/Hexagon.cpp (+17-3)
- (mo
llvmbot wrote:
@SidManning What do you think about merging this PR to the release branch?
https://github.com/llvm/llvm-project/pull/112040
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/112040
___
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/llvmbot created
https://github.com/llvm/llvm-project/pull/112040
Backport 77aa8257acbd773c0c430cd962da1bcfbd5ee94b
Requested by: @androm3da
>From 662e1b306be89ed7907cc1e19f24cf9e68ff7f0d Mon Sep 17 00:00:00 2001
From: Brian Cain
Date: Fri, 11 Oct 2024 14:31:41 -0500
Subject
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 5c25ba3380966ccac5bd0caf2c39bc10c1571e42
e88d91ffd1358f0c9cbd4fca14fc54a441fe2c36 --e
llvmbot wrote:
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: Vitaly Buka (vitalybuka)
Changes
There are hard to debug leaks which look like
false.
In general, repeating leak checking should not
affect set of leaks significantly, especial
`at_exit` leak checking.
But if we see signif
https://github.com/vitalybuka created
https://github.com/llvm/llvm-project/pull/112037
There are hard to debug leaks which look like
false.
In general, repeating leak checking should not
affect set of leaks significantly, especial
`at_exit` leak checking.
But if we see significant discrepancy,
https://github.com/efriedma-quic approved this pull request.
LGTM
I'm a little concerned about messing with datastructures in headers, but I
think, since the headers in question aren't exposed in clang/include, this
doesn't have any visible ABI effect.
Otherwise this is a pretty safe refactor
@@ -0,0 +1,73 @@
+//===-- sanitizer_thread_history.cpp
--===//
+//
+// 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,73 @@
+//===-- sanitizer_thread_history.cpp
--===//
+//
+// 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
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
perry-ca wrote:
I agree. By fixing it, I was referring to the code in clang that determines
this value. If it's easy to make the test pass on z/OS great. If not marking
it as unsupported works too. This applies to
https://github.com/ldionne commented:
LGTM thanks!
https://github.com/llvm/llvm-project/pull/109324
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
arsenm wrote:
The result should definitely be sm_70, that's what the provided script produces
https://github.com/llvm/llvm-project/pull/111976
___
llvm-branch-commits mailing
giordano wrote:
> and I had assumed from https://github.com/JuliaPackaging/Yggdrasil/pull/9204
> that this was related to pulling clang-19 into Julia.
LLVM, rather than Clang specifically, but yes, we're updating it to v19 for
Julia. Maybe @Zentrik can comment about how they got into #110382,
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
arsenm wrote:
This can't really be a property of zos or any other system. It works for me on
systems without AMDGPUs and other OSes. You are seeing the error after the
environment variable parse error, so why did this
Author: Janek van Oirschot
Date: 2024-10-11T17:01:32+01:00
New Revision: ebfd2950a25f5e8ff89e63c8c2cb7825a5a2d162
URL:
https://github.com/llvm/llvm-project/commit/ebfd2950a25f5e8ff89e63c8c2cb7825a5a2d162
DIFF:
https://github.com/llvm/llvm-project/commit/ebfd2950a25f5e8ff89e63c8c2cb7825a5a2d162.
uweigand wrote:
> @uweigand (or anyone else). If you would like to add a note about this fix in
> the release notes (completely optional). Please reply to this comment with a
> one or two sentence description of the fix. When you are done, please add the
> release:note label to this PR.
Fix a
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
perry-ca wrote:
This is failing on z/OS on line 28 (ARCH_sm70 check). The actual cc1 line is:
```
clang: warning: CUDA version is newer than the latest partially supported
version 12.5 [-Wunknown-cuda-version]
clang: e
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
perry-ca wrote:
This is failing on z/OS on line 33 with the error:
```
clang: error: failed to deduce triple for target architecture 'native'; specify
the triple using '-fopenmp-targets' and '-Xopenmp-target' instead
``
@@ -1,4 +1,3 @@
-// REQUIRES: system-linux
perry-ca wrote:
The compile & test on line 47 is failing on z/OS with the following message:
```
clang: error: cannot determine amdgcn architecture: No AMD GPU detected in the
system; consider passing it via '--offload-
https://github.com/perry-ca commented:
Three of these tests will fail on z/OS when the requires is removed. It would
be great if they can be fixed so they work on z/OS. None of these are
available or supported on z/OS so marking them as unsupported on system-zos is
also an option.
@daltenty
https://github.com/perry-ca edited
https://github.com/llvm/llvm-project/pull/111976
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
ldionne wrote:
I'm not seeing any CI failure anymore, I think this can be merged?
https://github.com/llvm/llvm-project/pull/110677
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/l
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/111984
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
AaronBallman wrote:
> @AaronBallman ok to backport?
Yeah, seems reasonable to me.
https://github.com/llvm/llvm-project/pull/111660
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
llvmbot wrote:
@llvm/pr-subscribers-llvm-transforms
Author: Yingwei Zheng (dtcxzyw)
Changes
Backport
https://github.com/llvm/llvm-project/commit/6a65e98fa7901dc1de91172d065fafb16ce89d77.
As https://github.com/llvm/llvm-project/pull/100899 exists in 19.x code base, I
guess 19.x is also a
https://github.com/dtcxzyw edited
https://github.com/llvm/llvm-project/pull/111984
___
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/dtcxzyw created
https://github.com/llvm/llvm-project/pull/111984
Backport
https://github.com/llvm/llvm-project/commit/6a65e98fa7901dc1de91172d065fafb16ce89d77.
As https://github.com/llvm/llvm-project/pull/100899 exists in 19.x code base, I
guess 19.x is also a vulnerable ve
https://github.com/dtcxzyw milestoned
https://github.com/llvm/llvm-project/pull/111984
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
Author: Matt Arsenault (arsenm)
Changes
The test only depends on AArch64 as it should.
---
Full diff: https://github.com/llvm/llvm-project/pull/111983.diff
1 Files Affected:
- (modified) llvm/test/CodeGen/AArch64/vecreduce-propagate-s
https://github.com/arsenm ready_for_review
https://github.com/llvm/llvm-project/pull/111983
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
arsenm wrote:
> [!WARNING]
> This pull request is not mergeable via GitHub because a downstack PR is
> open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/111983?utm_source=stack-comment-downstack-mergeability-warning";
https://github.com/arsenm created
https://github.com/llvm/llvm-project/pull/111983
The test only depends on AArch64 as it should.
>From b0146e190daaea20a9e5bae9f358590541729d6a Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Fri, 11 Oct 2024 15:16:30 +0400
Subject: [PATCH] AArch64: Remove
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Matt Arsenault (arsenm)
Changes
Works for me on macos.
---
Full diff: https://github.com/llvm/llvm-project/pull/111976.diff
7 Files Affected:
- (modified) clang/test/Driver/amdgpu-hip-system-arch.c (-1)
- (modified) clang/test/D
https://github.com/arsenm ready_for_review
https://github.com/llvm/llvm-project/pull/111976
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
arsenm wrote:
> [!WARNING]
> This pull request is not mergeable via GitHub because a downstack PR is
> open. Once all requirements are satisfied, merge this PR as a stack href="https://app.graphite.dev/github/pr/llvm/llvm-project/111976?utm_source=stack-comment-downstack-mergeability-warning";
https://github.com/arsenm created
https://github.com/llvm/llvm-project/pull/111976
Works for me on macos.
>From fee09ae97a755cb38961ba7cbd23e84409eeec9e Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Fri, 11 Oct 2024 14:33:32 +0400
Subject: [PATCH] clang: Remove requires system-linux from
@@ -112,42 +104,46 @@ entry:
define i32 @bcmp_size_2(ptr %s1, ptr %s2) nounwind optsize {
; CHECK-ALIGNED-RV32-LABEL: bcmp_size_2:
; CHECK-ALIGNED-RV32: # %bb.0: # %entry
-; CHECK-ALIGNED-RV32-NEXT:addi sp, sp, -16
-; CHECK-ALIGNED-RV32-NEXT:sw ra, 12(sp) # 4-byte
@@ -1144,42 +2872,116 @@ entry:
define i32 @memcmp_size_4(ptr %s1, ptr %s2) nounwind {
; CHECK-ALIGNED-RV32-LABEL: memcmp_size_4:
; CHECK-ALIGNED-RV32: # %bb.0: # %entry
-; CHECK-ALIGNED-RV32-NEXT:addi sp, sp, -16
-; CHECK-ALIGNED-RV32-NEXT:sw ra, 12(sp) # 4-byte
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/107548
>From f21cfcfc90330ee3856746b6315a81a00313b0e0 Mon Sep 17 00:00:00 2001
From: Wang Pengcheng
Date: Fri, 6 Sep 2024 17:20:51 +0800
Subject: [PATCH 1/5] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=
https://github.com/wangpc-pp updated
https://github.com/llvm/llvm-project/pull/107548
>From f21cfcfc90330ee3856746b6315a81a00313b0e0 Mon Sep 17 00:00:00 2001
From: Wang Pengcheng
Date: Fri, 6 Sep 2024 17:20:51 +0800
Subject: [PATCH 1/5] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/110827
___
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/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/111246
___
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/RKSimon approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/111796
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
mjklemm wrote:
Is this PR in an acceptable state to be merged? I went through the other three
and it seems that the 4 PRs all converging and seem either ready for merge or
very close.
https://github.com/llvm/llvm-project/pull/101446
___
llvm-branch-
phoebewang wrote:
LGTM.
https://github.com/llvm/llvm-project/pull/111246
___
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/frederick-vs-ja updated
https://github.com/llvm/llvm-project/pull/109324
>From 1360969b8125a633a3a8ad734c8a369bd3cf47c2 Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Thu, 19 Sep 2024 15:40:47 -0400
Subject: [PATCH] [libc++] Adjust the version of __cpp_lib_ranges in C++20
frederick-vs-ja wrote:
@ldionne I force-pushed the branch for resolving conflicts with #109291. Please
double-check the changes.
https://github.com/llvm/llvm-project/pull/109324
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
h
https://github.com/frederick-vs-ja updated
https://github.com/llvm/llvm-project/pull/109324
>From eb53b498045eca14b64897a329d24b182689fc14 Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Thu, 19 Sep 2024 15:40:47 -0400
Subject: [PATCH] [libc++] Adjust the version of __cpp_lib_ranges in C++20
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: None (llvmbot)
Changes
Backport e5b05a51b8151cc7788bbdea4d491e5ccfceedea
Requested by: @owenca
---
Full diff: https://github.com/llvm/llvm-project/pull/111958.diff
2 Files Affected:
- (modified) clang/lib/Format/UnwrappedLinePar
llvmbot wrote:
@mydeveloperday What do you think about merging this PR to the release branch?
https://github.com/llvm/llvm-project/pull/111958
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/111958
___
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/llvmbot created
https://github.com/llvm/llvm-project/pull/111958
Backport e5b05a51b8151cc7788bbdea4d491e5ccfceedea
Requested by: @owenca
>From c1946eec57dcc901211a7464e7893098cb252d50 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Fri, 4 Oct 2024 18:18:03 -0700
Subject: [PAT
Enna1 wrote:
Will uses of
[`__asan::DescribeThread`](https://github.com/llvm/llvm-project/blob/main/compiler-rt/lib/asan/asan_descriptions.cpp#L39)
be replaced with `__sanitizer::PrintThreadHistory` introduced in this patch?
https://github.com/llvm/llvm-project/pull/111948
65 matches
Mail list logo
