https://github.com/jurahul approved this pull request.
LGTM with a small comment
https://github.com/llvm/llvm-project/pull/141762
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/ll
@@ -113,14 +113,39 @@ class BaseRecord {
// Returns the name of the directive formatted for output. Whitespace are
// replaced with underscores.
- static std::string formatName(StringRef Name) {
+ static std::string getSnakeName(StringRef Name) {
std::string N = Nam
https://github.com/Endilll commented:
Something feels wrong with this PR:
1) There's a "full changes here" button on the Files Changed tab, which shows
me a bunch of changes to libc++ tests adding `#include `
2) In the new version, I don't see `LLVM_ENABLE_RUTIMES` passed to cmake. Not
even `CM
https://github.com/aaupov updated
https://github.com/llvm/llvm-project/pull/142698
___
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/142698
___
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.
https://github.com/llvm/llvm-project/pull/142698
___
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/qinkunbao edited
https://github.com/llvm/llvm-project/pull/142701
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -1030,6 +1036,11 @@ bool MachineBlockPlacement::isTrellis(
SmallPtrSet SeenPreds;
for (MachineBasicBlock *Succ : ViableSuccs) {
+// Compile-time optimization: runtime is quadratic in the number of
+// predecessors. For such uncommon cases, exit early.
+if (S
Author: Owen Pan
Date: 2025-06-03T15:37:21-07:00
New Revision: f6532710ace867e5660ea9e47ab77f2b8349896f
URL:
https://github.com/llvm/llvm-project/commit/f6532710ace867e5660ea9e47ab77f2b8349896f
DIFF:
https://github.com/llvm/llvm-project/commit/f6532710ace867e5660ea9e47ab77f2b8349896f.diff
LOG:
@@ -216,16 +227,16 @@ def get_env_variables(modified_files: list[str],
platform: str) -> Set[str]:
projects_to_test = _compute_projects_to_test(modified_projects, platform)
projects_to_build = _compute_projects_to_build(projects_to_test)
projects_check_targets = _c
https://github.com/arsenm created
https://github.com/llvm/llvm-project/pull/142708
This appears to only be useful for the eq/ne cases, and only for
ARM libcalls. There's no change in the new fcmp test output.
>From e4b22e5ee9c79d2f50724147f67bfd09ad264478 Mon Sep 17 00:00:00 2001
From: Matt Ars
@@ -216,16 +227,16 @@ def get_env_variables(modified_files: list[str],
platform: str) -> Set[str]:
projects_to_test = _compute_projects_to_test(modified_projects, platform)
projects_to_build = _compute_projects_to_build(projects_to_test)
projects_check_targets = _c
https://github.com/arsenm ready_for_review
https://github.com/llvm/llvm-project/pull/142708
___
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-msp430
Author: Matt Arsenault (arsenm)
Changes
This appears to only be useful for the eq/ne cases, and only for
ARM libcalls. There's no change in the new fcmp test output.
---
Full diff: https://github.com/llvm/llvm-project/pull/142708.diff
1
@@ -216,16 +227,16 @@ def get_env_variables(modified_files: list[str],
platform: str) -> Set[str]:
projects_to_test = _compute_projects_to_test(modified_projects, platform)
projects_to_build = _compute_projects_to_build(projects_to_test)
projects_check_targets = _c
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/142708?utm_source=stack-comment-downstack-mergeability-warning";
qinkunbao wrote:
Close this PR because of https://github.com/llvm/llvm-project/pull/142659
https://github.com/llvm/llvm-project/pull/142701
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
https://github.com/aaupov edited
https://github.com/llvm/llvm-project/pull/142698
___
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/qinkunbao closed
https://github.com/llvm/llvm-project/pull/142701
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
Author: Teresa Johnson
Date: 2025-06-03T16:04:38-07:00
New Revision: ed2f4fb23451c01cc86c23f5ccc9fc2d0d1427ff
URL:
https://github.com/llvm/llvm-project/commit/ed2f4fb23451c01cc86c23f5ccc9fc2d0d1427ff
DIFF:
https://github.com/llvm/llvm-project/commit/ed2f4fb23451c01cc86c23f5ccc9fc2d0d1427ff.diff
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/142701
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
dianqk wrote:
Note to me: I propose to keep a major version around until the gnu linker
release supports it and GitHub Actions defaults to Xcode 15.3+.
https://github.com/llvm/llvm-project/pull/142311
___
llvm-branch-commits mailing list
llvm-branch-c
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Qinkun Bao (qinkunbao)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142701.diff
2 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+3-1)
- (modified) clang/docs/SanitizerSpecialCaseList.rst (+16-2)
https://github.com/qinkunbao created
https://github.com/llvm/llvm-project/pull/142701
None
___
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
pre-aggregated traces. Fix by zero initializing both counters.
Test Plan: updated entry-point-fallthru.s
---
Full diff: https://github.com/llvm/llvm-project/pull/142698.diff
2 Files Affected:
- (modified) b
https://github.com/aaupov created
https://github.com/llvm/llvm-project/pull/142698
pre-aggregated traces. Fix by zero initializing both counters.
Test Plan: updated entry-point-fallthru.s
___
llvm-branch-commits mailing list
llvm-branch-commits@lis
dianqk wrote:
> @dianqk (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.
Added a w
https://github.com/boomanaiden154 created
https://github.com/llvm/llvm-project/pull/142695
This patch adjusts the compute_projects script to explicitly determine
what runtimes should be built and what runtimes should be tested. This
is mainly to support enabling runtimes for LLDB testing but not
boomanaiden154 wrote:
This/dependent patches are intended to land after we deprecate/shutdown
Buildkite. No effort was made here to keep the Buildkite pipeline working in
the same manner as the Github workflow.
https://github.com/llvm/llvm-project/pull/142696
__
llvmbot wrote:
@llvm/pr-subscribers-github-workflow
Author: Aiden Grossman (boomanaiden154)
Changes
This patch migrates the premerge pipeline to use LLVM_ENABLE_RUNTIMES to
build libc and compiler-rt.
---
Full diff: https://github.com/llvm/llvm-project/pull/142696.diff
4 Files Affected
https://github.com/boomanaiden154 created
https://github.com/llvm/llvm-project/pull/142696
This patch migrates the premerge pipeline to use LLVM_ENABLE_RUNTIMES to
build libc and compiler-rt.
___
llvm-branch-commits mailing list
llvm-branch-commits@
https://github.com/boomanaiden154 created
https://github.com/llvm/llvm-project/pull/142694
This patch switches us to using LLVM_ENABLE_RUNTIMES rather than using
separate runtimes builds for some reductions in CMake configuration time
and some simplification of the monolithic-linux.sh script.
https://github.com/tstellar milestoned
https://github.com/llvm/llvm-project/pull/142648
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
github-actions[bot] wrote:
@owenca (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.
ht
https://github.com/tstellar closed
https://github.com/llvm/llvm-project/pull/142482
___
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 updated
https://github.com/llvm/llvm-project/pull/142482
>From f6532710ace867e5660ea9e47ab77f2b8349896f Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Mon, 2 Jun 2025 13:35:27 -0700
Subject: [PATCH] [clang-format] Correctly annotate token-pasted function decl
names (#
@@ -57,7 +57,8 @@ class AArch64FrameLowering : public TargetFrameLowering {
StackOffset resolveFrameOffsetReference(const MachineFunction &MF,
int64_t ObjectOffset, bool isFixed,
bool isSVE, R
@@ -3780,25 +3938,49 @@ void AArch64FrameLowering::determineStackHazardSlot(
bool HasFPRCSRs = any_of(SavedRegs.set_bits(), [](unsigned Reg) {
return AArch64::FPR64RegClass.contains(Reg) ||
AArch64::FPR128RegClass.contains(Reg) ||
- AArch64::ZPRRegCla
https://github.com/efriedma-quic commented:
If you express the size of the hazard padding between the PPRs and ZPRs as a
scalable size, that might simplify some of the logic? You wouldn't need to
represent the two areas as separate stacks, at least.
Maybe it's also worth considering "prealloc
https://github.com/efriedma-quic edited
https://github.com/llvm/llvm-project/pull/142392
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
github-actions[bot] wrote:
@dianqk (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.
ht
https://github.com/tstellar closed
https://github.com/llvm/llvm-project/pull/142311
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
Author: dianqk
Date: 2025-06-03T14:49:03-07:00
New Revision: e0586e278f96e4f104c8b8039a41c6832c35be22
URL:
https://github.com/llvm/llvm-project/commit/e0586e278f96e4f104c8b8039a41c6832c35be22
DIFF:
https://github.com/llvm/llvm-project/commit/e0586e278f96e4f104c8b8039a41c6832c35be22.diff
LOG: [
https://github.com/tstellar updated
https://github.com/llvm/llvm-project/pull/142311
>From e0586e278f96e4f104c8b8039a41c6832c35be22 Mon Sep 17 00:00:00 2001
From: dianqk
Date: Sun, 1 Jun 2025 22:17:57 +0800
Subject: [PATCH] [RelLookupTableConverter] Drop unnamed_addr to avoid
generating GOTPCR
https://github.com/tstellar closed
https://github.com/llvm/llvm-project/pull/141957
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
Author: Matheus Izvekov
Date: 2025-05-29T11:43:19-03:00
New Revision: 7759bb57c24390797ee34fa58a5e1234f5aa9369
URL:
https://github.com/llvm/llvm-project/commit/7759bb57c24390797ee34fa58a5e1234f5aa9369
DIFF:
https://github.com/llvm/llvm-project/commit/7759bb57c24390797ee34fa58a5e1234f5aa9369.dif
https://github.com/rockwotj commented:
Thank you!
https://github.com/llvm/llvm-project/pull/142648
___
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/bhandarkar-pranav updated
https://github.com/llvm/llvm-project/pull/141715
>From bc9fbf66365fbde456813ff9c4354fb378ee45cf Mon Sep 17 00:00:00 2001
From: Pranav Bhandarkar
Date: Fri, 23 May 2025 10:26:14 -0500
Subject: [PATCH 1/5] Fix boxchar with firstprivate
---
.../Optimi
@@ -743,6 +800,37 @@ class MapInfoFinalizationPass
return mlir::WalkResult::advance();
});
+ func->walk([&](mlir::omp::MapInfoOp op) {
+if (!op.getMembers().empty())
+ return;
+
+if (!mlir::isa(fir::unwrapRefType(op.getVarType(
+
@@ -285,6 +285,62 @@ class MapInfoFinalizationPass
return false;
}
+ mlir::omp::MapInfoOp genBoxcharMemberMap(mlir::omp::MapInfoOp op,
agozillon wrote:
Probably a dumb question, but wouldn't this and the related walk already be
handled by the larger:
https://github.com/agozillon approved this pull request.
This LGTM, just had one question that's all :-)
https://github.com/llvm/llvm-project/pull/141715
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cg
https://github.com/agozillon edited
https://github.com/llvm/llvm-project/pull/141715
___
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/joaosaffran updated
https://github.com/llvm/llvm-project/pull/142492
>From 3e6b07e119988058defd305199ad3e08d424eebd Mon Sep 17 00:00:00 2001
From: joaosaffran
Date: Mon, 2 Jun 2025 19:36:32 +
Subject: [PATCH 1/6] add parsing
---
.../BinaryFormat/DXContainerConstants.def
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 HEAD~1 HEAD --extensions cpp,h --
llvm/lib/Target/DirectX/DXILRootSignature.cpp
llvm
https://github.com/joaosaffran updated
https://github.com/llvm/llvm-project/pull/142492
>From 3e6b07e119988058defd305199ad3e08d424eebd Mon Sep 17 00:00:00 2001
From: joaosaffran
Date: Mon, 2 Jun 2025 19:36:32 +
Subject: [PATCH 1/5] add parsing
---
.../BinaryFormat/DXContainerConstants.def
https://github.com/momchil-velikov updated
https://github.com/llvm/llvm-project/pull/142422
>From 1590fe674c00e07171c8842805a06907ed68f242 Mon Sep 17 00:00:00 2001
From: Momchil Velikov
Date: Mon, 2 Jun 2025 15:13:13 +
Subject: [PATCH 1/2] [MLIR] Fix incorrect slice contiguity inference in
https://github.com/bhandarkar-pranav updated
https://github.com/llvm/llvm-project/pull/141715
>From bc9fbf66365fbde456813ff9c4354fb378ee45cf Mon Sep 17 00:00:00 2001
From: Pranav Bhandarkar
Date: Fri, 23 May 2025 10:26:14 -0500
Subject: [PATCH 1/4] Fix boxchar with firstprivate
---
.../Optimi
https://github.com/efriedma-quic created
https://github.com/llvm/llvm-project/pull/142648
Backport 97885213bd4507b204b050c3cd570e365d21cc7d
>From c40a997d92a6939e3fb75c6a8766c851c57f3c58 Mon Sep 17 00:00:00 2001
From: Eli Friedman
Date: Tue, 3 Jun 2025 09:51:37 -0700
Subject: [PATCH] release/2
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Eli Friedman (efriedma-quic)
Changes
Backport 97885213bd4507b204b050c3cd570e365d21cc7d
---
Full diff: https://github.com/llvm/llvm-project/pull/142648.diff
2 Files Affected:
- (modified) clang/lib/AST/ExprConstant.cpp (+6-1)
- (modifie
Author: Ian Wood
Date: 2025-06-03T10:30:22-07:00
New Revision: b3e60ba212782619b5c8c12542a4d9ecd539a652
URL:
https://github.com/llvm/llvm-project/commit/b3e60ba212782619b5c8c12542a4d9ecd539a652
DIFF:
https://github.com/llvm/llvm-project/commit/b3e60ba212782619b5c8c12542a4d9ecd539a652.diff
LOG:
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/142468
___
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/vitalybuka updated
https://github.com/llvm/llvm-project/pull/142468
___
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/joaosaffran updated
https://github.com/llvm/llvm-project/pull/142492
>From 3e6b07e119988058defd305199ad3e08d424eebd Mon Sep 17 00:00:00 2001
From: joaosaffran
Date: Mon, 2 Jun 2025 19:36:32 +
Subject: [PATCH 1/5] add parsing
---
.../BinaryFormat/DXContainerConstants.def
https://github.com/momchil-velikov updated
https://github.com/llvm/llvm-project/pull/142422
>From 1590fe674c00e07171c8842805a06907ed68f242 Mon Sep 17 00:00:00 2001
From: Momchil Velikov
Date: Mon, 2 Jun 2025 15:13:13 +
Subject: [PATCH 1/2] [MLIR] Fix incorrect slice contiguity inference in
Author: Ian Wood
Date: 2025-06-03T09:46:37-07:00
New Revision: 8b407ad6fcb1e722d744edb41dac39bf94117b8a
URL:
https://github.com/llvm/llvm-project/commit/8b407ad6fcb1e722d744edb41dac39bf94117b8a
DIFF:
https://github.com/llvm/llvm-project/commit/8b407ad6fcb1e722d744edb41dac39bf94117b8a.diff
LOG:
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/142468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -849,29 +869,26 @@ void PromoteMem2Reg::run() {
// Set the incoming values for the basic block to be null values for all of
// the alloca's. We do this in case there is a load of a value that has not
// been stored yet. In this case, it will get this null value.
- R
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/142468
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
MacDue wrote:
Rebased this PR stack on the changes from #138609... Which makes things even
hairier :sweat_smile: It would be nice if all these modes were not so
intertwined in the code.
https://github.com/llvm/llvm-project/pull/142392
___
llvm-bran
https://github.com/MacDue updated
https://github.com/llvm/llvm-project/pull/142391
>From 42af819d001699ae6361d51e34d76a06fe956250 Mon Sep 17 00:00:00 2001
From: Benjamin Maxwell
Date: Thu, 8 May 2025 17:38:27 +
Subject: [PATCH] [AArch64] Prepare for split ZPR and PPR area allocation
(NFCI)
Author: Orlando Cazalet-Hyams
Date: 2025-06-03T16:15:12+01:00
New Revision: 36efd649134e82ec7083c1fd02ff6f11b128604f
URL:
https://github.com/llvm/llvm-project/commit/36efd649134e82ec7083c1fd02ff6f11b128604f
DIFF:
https://github.com/llvm/llvm-project/commit/36efd649134e82ec7083c1fd02ff6f11b12860
@@ -2,6 +2,8 @@
# RUN: llc -mtriple=amdgcn -run-pass=regbankselect %s -verify-machineinstrs -o
- -regbankselect-fast | FileCheck %s
# RUN: llc -mtriple=amdgcn -run-pass=regbankselect %s -verify-machineinstrs -o
- -regbankselect-greedy | FileCheck %s
+# RUN: llc -mtriple=amdg
petar-avramovic wrote:
This one LGTM, minus patches before
https://github.com/llvm/llvm-project/pull/142602
___
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/arsenm ready_for_review
https://github.com/llvm/llvm-project/pull/142617
___
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-arm
Author: Matt Arsenault (arsenm)
Changes
These Module level triple checks implemented in the Subtarget are kind of
a pain and we should probably get rid of all of them in across all targets,
and stick to a consistent set of names.
---
Full di
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/142617?utm_source=stack-comment-downstack-mergeability-warning";
https://github.com/arsenm created
https://github.com/llvm/llvm-project/pull/142617
These Module level triple checks implemented in the Subtarget are kind of
a pain and we should probably get rid of all of them in across all targets,
and stick to a consistent set of names.
>From cf2ad5a269d23208
arsenm wrote:
> These are mostly used for loads/stores and atomics, is it really that simple,
> all 32bit pointer types should have same rules?
Yes
https://github.com/llvm/llvm-project/pull/142600
___
llvm-branch-commits mailing list
llvm-branch-comm
petar-avramovic wrote:
Don't like adding all these without use case.
Not sure if we can get away with only checking ptr size and ignoring address
space but in the end it is mostly used for assert.
These are mostly used for loads/stores and atomics, is it really that simple,
all 32bit pointer t
https://github.com/arsenm commented:
We shouldn't need to treat every address space as unique, just 32 or 64 bit
https://github.com/llvm/llvm-project/pull/142600
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llv
tgymnich wrote:
maybe add tests?
https://github.com/llvm/llvm-project/pull/142600
___
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/tgymnich approved this pull request.
https://github.com/llvm/llvm-project/pull/142600
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
OCHyams wrote:
Landed in 8e50e882a847a8fd51ee8ec22fea5df192b14c06
https://github.com/llvm/llvm-project/pull/141618
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-comm
OCHyams wrote:
Nope that didn't seem to work. I'll land it manually...
> will destructors that run when "break"ing out of a loop get the key-ness of
> the "break" statement? (This might not be a bad thing at all).
Similar answer to elsewhere - calls aren't annotated by the front end, instead
https://github.com/Pierre-vh updated
https://github.com/llvm/llvm-project/pull/142604
>From f9141055a09fc376354f901932b0a39e28d58c35 Mon Sep 17 00:00:00 2001
From: pvanhout
Date: Tue, 3 Jun 2025 15:08:06 +0200
Subject: [PATCH 1/2] [AMDGPU] New RegBankSelect: Add rules for `G_PTRTOINT`
and `G_I
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 HEAD~1 HEAD --extensions cpp --
llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeRules.cpp
OCHyams wrote:
I didn't close this, GitHub did (because I merged the base branch? for others
that resulted in rebasing dependant branches on main... I don't understand why
this one was different). I don't seem to be able to reopen it or convert it to
a draft since the base branch no longer exi
llvmbot wrote:
@llvm/pr-subscribers-llvm-globalisel
Author: Pierre van Houtryve (Pierre-vh)
Changes
Test P0 through P6 + P8 for both S/VGPRs.
---
Full diff: https://github.com/llvm/llvm-project/pull/142603.diff
2 Files Affected:
- (modified) llvm/test/CodeGen/AMDGPU/GlobalISel/regbanks
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Pierre van Houtryve (Pierre-vh)
Changes
Test P0 through P6 + P8 for both S/VGPRs.
---
Full diff: https://github.com/llvm/llvm-project/pull/142603.diff
2 Files Affected:
- (modified) llvm/test/CodeGen/AMDGPU/GlobalISel/regbankse
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Pierre van Houtryve (Pierre-vh)
Changes
There's quite a few opcodes that do not care about the exact AS of the pointer,
just its size.
Adding generic types for these will help reduce duplication in the rule
definitions.
I also m
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Pierre van Houtryve (Pierre-vh)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142604.diff
3 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeRules.cpp (+15-1)
- (modified) llvm/test/
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Pierre van Houtryve (Pierre-vh)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142601.diff
3 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp (+9)
- (modified) llvm/lib/Tar
llvmbot wrote:
@llvm/pr-subscribers-backend-amdgpu
Author: Pierre van Houtryve (Pierre-vh)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142600.diff
3 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeHelper.cpp (+31)
- (modified) llvm/lib/Ta
llvmbot wrote:
@llvm/pr-subscribers-llvm-globalisel
Author: Pierre van Houtryve (Pierre-vh)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/142604.diff
3 Files Affected:
- (modified) llvm/lib/Target/AMDGPU/AMDGPURegBankLegalizeRules.cpp (+15-1)
- (modified) llvm/test
https://github.com/Pierre-vh ready_for_review
https://github.com/llvm/llvm-project/pull/142602
___
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/Pierre-vh ready_for_review
https://github.com/llvm/llvm-project/pull/142604
___
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/Pierre-vh ready_for_review
https://github.com/llvm/llvm-project/pull/142601
___
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/Pierre-vh ready_for_review
https://github.com/llvm/llvm-project/pull/142603
___
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/Pierre-vh ready_for_review
https://github.com/llvm/llvm-project/pull/142600
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
Pierre-vh 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/142604?utm_source=stack-comment-downstack-mergeability-warning
1 - 100 of 124 matches
Mail list logo
