https://github.com/SixWeining closed
https://github.com/llvm/llvm-project/pull/106332
___
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/heiher approved this pull request.
https://github.com/llvm/llvm-project/pull/106332
___
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,102 @@
+; RUN: opt -S -dxil-op-lower %s | FileCheck %s
+
+target triple = "dxil-pc-shadermodel6.6-compute"
+
+declare void @scalar_user(float)
+declare void @vector_user(<4 x float>)
+
+define void @loadfloats() {
+ ; CHECK: [[BIND:%.*]] = call %dx.types.Handle @dx.op.
@@ -0,0 +1,102 @@
+; RUN: opt -S -dxil-op-lower %s | FileCheck %s
+
+target triple = "dxil-pc-shadermodel6.6-compute"
+
+declare void @scalar_user(float)
+declare void @vector_user(<4 x float>)
+
+define void @loadfloats() {
+ ; CHECK: [[BIND:%.*]] = call %dx.types.Handle @dx.op.
@@ -0,0 +1,102 @@
+; RUN: opt -S -dxil-op-lower %s | FileCheck %s
+
+target triple = "dxil-pc-shadermodel6.6-compute"
+
+declare void @scalar_user(float)
+declare void @vector_user(<4 x float>)
+
+define void @loadfloats() {
+ ; CHECK: [[BIND:%.*]] = call %dx.types.Handle @dx.op.
@@ -236,6 +236,59 @@ class OpLowerer {
lowerToBindAndAnnotateHandle(F);
}
+ void lowerTypedBufferLoad(Function &F) {
+IRBuilder<> &IRB = OpBuilder.getIRB();
+Type *Int32Ty = IRB.getInt32Ty();
+
+replaceFunction(F, [&](CallInst *CI) -> Error {
+ IRB.Se
@@ -683,6 +686,17 @@ def CreateHandle : DXILOp<57, createHandle> {
let stages = [Stages];
}
+def BufferLoad : DXILOp<68, bufferLoad> {
+ let Doc = "reads from a TypedBuffer";
+ // Handle, Coord0, Coord1
+ let arguments = [HandleTy, Int32Ty, Int32Ty];
+ let result = Overl
https://github.com/bogner created
https://github.com/llvm/llvm-project/pull/106645
This is a follow up to #104252 that returns a struct instead of a vector. If we
decide this is better, I'll presumably include the change in that PR rather
than actually doing a separate follow up.
@@ -932,18 +935,55 @@ bool
MachineCSE::isProfitableToHoistInto(MachineBasicBlock *CandidateBB,
MBFI->getBlockFreq(MBB) + MBFI->getBlockFreq(MBB1);
}
-bool MachineCSE::runOnMachineFunction(MachineFunction &MF) {
- if (skipFunction(MF.getFunction()))
-return false
https://github.com/cdevadas updated
https://github.com/llvm/llvm-project/pull/106605
>From 607099de09be2fed6d9277c8439ade69e0820d92 Mon Sep 17 00:00:00 2001
From: Christudasan Devadasan
Date: Thu, 29 Aug 2024 22:21:22 +0530
Subject: [PATCH 1/3] [CodeGen][NewPM] Port MachineCSE pass to new pass
@@ -932,18 +935,55 @@ bool
MachineCSE::isProfitableToHoistInto(MachineBasicBlock *CandidateBB,
MBFI->getBlockFreq(MBB) + MBFI->getBlockFreq(MBB1);
}
-bool MachineCSE::runOnMachineFunction(MachineFunction &MF) {
- if (skipFunction(MF.getFunction()))
-return false
@@ -932,18 +935,55 @@ bool
MachineCSE::isProfitableToHoistInto(MachineBasicBlock *CandidateBB,
MBFI->getBlockFreq(MBB) + MBFI->getBlockFreq(MBB1);
}
-bool MachineCSE::runOnMachineFunction(MachineFunction &MF) {
- if (skipFunction(MF.getFunction()))
-return false
@@ -0,0 +1,29 @@
+//===- llvm/CodeGen/MachineCSE.h -*- C++ -*-===//
arsenm wrote:
Should pad out to 80 lines
https://github.com/llvm/llvm-project/pull/106605
___
llvm-branch-commits mailing list
llvm-b
@@ -932,18 +935,55 @@ bool
MachineCSE::isProfitableToHoistInto(MachineBasicBlock *CandidateBB,
MBFI->getBlockFreq(MBB) + MBFI->getBlockFreq(MBB1);
}
-bool MachineCSE::runOnMachineFunction(MachineFunction &MF) {
- if (skipFunction(MF.getFunction()))
-return false
https://github.com/cdevadas updated
https://github.com/llvm/llvm-project/pull/106605
>From 607099de09be2fed6d9277c8439ade69e0820d92 Mon Sep 17 00:00:00 2001
From: Christudasan Devadasan
Date: Thu, 29 Aug 2024 22:21:22 +0530
Subject: [PATCH 1/2] [CodeGen][NewPM] Port MachineCSE pass to new pass
llvmbot wrote:
@llvm/pr-subscribers-backend-nvptx
@llvm/pr-subscribers-backend-amdgpu
Author: Christudasan Devadasan (cdevadas)
Changes
---
Patch is 33.28 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/106605.diff
21 Files Affected:
- (add
https://github.com/cdevadas ready_for_review
https://github.com/llvm/llvm-project/pull/106605
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
cdevadas 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/106605?utm_source=stack-comment-downstack-mergeability-warning"
https://github.com/cdevadas created
https://github.com/llvm/llvm-project/pull/106605
None
>From 607099de09be2fed6d9277c8439ade69e0820d92 Mon Sep 17 00:00:00 2001
From: Christudasan Devadasan
Date: Thu, 29 Aug 2024 22:21:22 +0530
Subject: [PATCH] [CodeGen][NewPM] Port MachineCSE pass to new pas
https://github.com/ahmedbougacha milestoned
https://github.com/llvm/llvm-project/pull/106599
___
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/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/106599
___
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/ahmedbougacha edited
https://github.com/llvm/llvm-project/pull/106599
___
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: Ahmed Bougacha (ahmedbougacha)
Changes
This is a partial revert of c66e1d6f3429. Even though that allowed us to
declare v9.2-a support without picking up SVE2 in both the backend and the
driver, the frontend itself still enable
https://github.com/ahmedbougacha created
https://github.com/llvm/llvm-project/pull/106599
This is a partial revert of c66e1d6f3429. Even though that allowed us to
declare v9.2-a support without picking up SVE2 in both the backend and the
driver, the frontend itself still enabled SVE via the a
Author: Mehdi Amini
Date: 2024-08-29T10:29:20-07:00
New Revision: 89a7a0b178e406164641e3f88bd5694994b15ba8
URL:
https://github.com/llvm/llvm-project/commit/89a7a0b178e406164641e3f88bd5694994b15ba8
DIFF:
https://github.com/llvm/llvm-project/commit/89a7a0b178e406164641e3f88bd5694994b15ba8.diff
L
s-barannikov wrote:
> I don't think this is the right direction for memset. I would prefer actually
> overloading it and making the memset/memset.pattern distinction redundant.
> See also [#97583
> (comment)](https://github.com/llvm/llvm-project/pull/97583#issuecomment-2288302946)
> and follo
https://github.com/mizvekov approved this pull request.
https://github.com/llvm/llvm-project/pull/106516
___
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/5chmidti updated
https://github.com/llvm/llvm-project/pull/105935
>From 2072160d9b7763c58edd14083a4523fd94e6040f Mon Sep 17 00:00:00 2001
From: Julian Schmidt
Date: Sat, 1 Jun 2024 17:49:13 +0200
Subject: [PATCH] [clang][test] remove unused `run` overload in
`BoundNodesCallb
Author: Tom Eccles
Date: 2024-08-29T15:35:01+01:00
New Revision: fd0affbd96b3e243880c31212b590db381e349c6
URL:
https://github.com/llvm/llvm-project/commit/fd0affbd96b3e243880c31212b590db381e349c6
DIFF:
https://github.com/llvm/llvm-project/commit/fd0affbd96b3e243880c31212b590db381e349c6.diff
LO
ivanradanov wrote:
> Can you share a case where this would happen? I agree that we wouldn't want
> to produce some IR that doesn't keep consistent semantics for a given
> operation across the pipeline. In that case, adding another operation might
> indeed be the right solution.
I was under t
@@ -2193,80 +2197,141 @@ llvm::Value *getSizeInBytes(DataLayout &dl, const
mlir::Type &type,
return builder.getInt64(dl.getTypeSizeInBits(type) / 8);
}
-void collectMapDataFromMapVars(MapInfoData &mapData,
- llvm::SmallVectorImpl &mapVars,
-
@@ -2193,80 +2197,137 @@ llvm::Value *getSizeInBytes(DataLayout &dl, const
mlir::Type &type,
return builder.getInt64(dl.getTypeSizeInBits(type) / 8);
}
-void collectMapDataFromMapVars(MapInfoData &mapData,
- llvm::SmallVectorImpl &mapVars,
-
https://github.com/skatrak approved this pull request.
Thank you, LGTM. Just a small comment left from me.
https://github.com/llvm/llvm-project/pull/101707
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/skatrak edited
https://github.com/llvm/llvm-project/pull/101707
___
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 commented:
For this and https://github.com/llvm/llvm-project/pull/106540, I'm not
convinced that we should actually make the canonical `i8` type for ptradd byte
width dependent. The only thing that matters is that the used type has an alloc
size of one byte, so always
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/106546
___
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/106546
Backport 2a28df66dc3f7ff5b6233241837854acefb68d77
Requested by: @zmodem
>From 10029e192d1c563a13eaea0171ad3924f0d6b4d0 Mon Sep 17 00:00:00 2001
From: Hans
Date: Thu, 29 Aug 2024 13:54:30 +0200
Subject: [PATCH]
https://github.com/nikic commented:
I don't think this is the right direction for memset. I would prefer actually
overloading it and making the memset/memset.pattern distinction redundant. See
also https://github.com/llvm/llvm-project/pull/97583#issuecomment-2288302946
and following comments.
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 2a28c47e53009b0ab5506deae34176d56b95e8c2
7cca42199cf0bae1b5648ff6b1dcc42677001548 --e
s-barannikov 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/106542?utm_source=stack-comment-downstack-mergeability-warn
https://github.com/s-barannikov created
https://github.com/llvm/llvm-project/pull/106542
The patch makes CharWidth argument of `getStringLength` mandatory
and ensures the correct values are passed in most cases.
This is *not* a complete support for unusual byte widths in
SimplifyLibCalls since `
s-barannikov 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/106541?utm_source=stack-comment-downstack-mergeability-warn
s-barannikov 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/106540?utm_source=stack-comment-downstack-mergeability-warn
https://github.com/s-barannikov created
https://github.com/llvm/llvm-project/pull/106541
The method assumes that host chars and target chars have the same width.
Add a CharWidth argument so that it can bail out if the requested char
width differs from the host char width.
Alternatively, the che
s-barannikov 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/106539?utm_source=stack-comment-downstack-mergeability-warn
https://github.com/s-barannikov created
https://github.com/llvm/llvm-project/pull/106540
The method has few uses yet, so just pass DL argument to it. The change
follows m_PtrToIntSameSize, and I don't see a better way of delivering
the byte width to the method.
>From 7cca42199cf0bae1b5648ff6b1d
s-barannikov 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/106538?utm_source=stack-comment-downstack-mergeability-warn
s-barannikov 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/106537?utm_source=stack-comment-downstack-mergeability-warn
https://github.com/s-barannikov created
https://github.com/llvm/llvm-project/pull/106539
The change requires DataLayout instance to be available, which, in turn,
requires insertion point to be set. In-tree tests detected only one case
when the function was called without setting an insertion poi
https://github.com/s-barannikov created
https://github.com/llvm/llvm-project/pull/106537
This patch changes the type of the value argument of @llvm.memset and
similar intrinsics from i8 to iN, where N is the byte width specified
in data layout string.
Note that the argument still has fixed type
https://github.com/s-barannikov created
https://github.com/llvm/llvm-project/pull/106538
This is a simple change to show how easy it can be to support unusual
byte widths in the middle end.
>From 3b86337919c0f518c133f09787ddc0518ede4821 Mon Sep 17 00:00:00 2001
From: Sergei Barannikov
Date: Fr
cor3ntin wrote:
we did not add one (because there was already a PR that partially but not fully
addressed this bug)
https://github.com/llvm/llvm-project/pull/106516
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.
https://github.com/tamird approved this pull request.
https://github.com/llvm/llvm-project/pull/106525
___
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/AaronBallman approved this pull request.
LGTM! Do we have a release note we need to remove as well?
https://github.com/llvm/llvm-project/pull/106516
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://list
TIFitis wrote:
Ping for review.
https://github.com/llvm/llvm-project/pull/101707
___
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-llvm-support
Author: None (llvmbot)
Changes
Backport fcb3a048
Requested by: @tamird
---
Full diff: https://github.com/llvm/llvm-project/pull/106525.diff
1 Files Affected:
- (modified) llvm/lib/Support/Z3Solver.cpp (+1)
``diff
diff --git a
https://github.com/llvmbot created
https://github.com/llvm/llvm-project/pull/106525
Backport fcb3a048
Requested by: @tamird
>From 6c9ea06ea23c369eee40dc88ebe677c462fcbe19 Mon Sep 17 00:00:00 2001
From: Luke Shingles
Date: Thu, 29 Aug 2024 11:09:07 +0100
Subject: [PATCH] [analyzer] Add missing
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/106525
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
llvmbot wrote:
@tamird What do you think about merging this PR to the release branch?
https://github.com/llvm/llvm-project/pull/106525
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/100389
>From 9061a141b8c2d39a474cf9e347ab2e47dabbde2d Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Wed, 1 Feb 2023 09:06:59 -0400
Subject: [PATCH] DAG: Lower fcNormal is.fpclass to compare with inf
Looks worse f
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/100380
>From f4df5b37e4f2f493a75be4b971aea5d64e6dff0b Mon Sep 17 00:00:00 2001
From: Matt Arsenault
Date: Wed, 1 Feb 2023 09:52:34 -0400
Subject: [PATCH] DAG: Lower single infinity is.fpclass tests to fcmp
InstCombine
arsenm wrote:
### Merge activity
* **Aug 29, 5:54 AM EDT**: @arsenm started a stack merge that includes this
pull request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/105577).
https://github.com/llvm/llvm-project/pull/105577
https://github.com/skatrak updated
https://github.com/llvm/llvm-project/pull/106211
>From 3508bc032e4fe208f1dc58f0fd979cf769f6ae17 Mon Sep 17 00:00:00 2001
From: Sergio Afonso
Date: Tue, 27 Aug 2024 12:51:25 +0100
Subject: [PATCH] [Flang][OpenMP] DISTRIBUTE PARALLEL DO SIMD lowering
This patch
https://github.com/skatrak updated
https://github.com/llvm/llvm-project/pull/106207
>From 1ae4c05cbaf186df1805314e213c733421bd1949 Mon Sep 17 00:00:00 2001
From: Sergio Afonso
Date: Tue, 27 Aug 2024 11:14:14 +0100
Subject: [PATCH] [Flang][OpenMP] DISTRIBUTE PARALLEL DO lowering
This patch adds
https://github.com/skatrak updated
https://github.com/llvm/llvm-project/pull/106066
>From c28e6de6fca8fe9142045dc81d5aa48e821524df Mon Sep 17 00:00:00 2001
From: Sergio Afonso
Date: Mon, 26 Aug 2024 13:29:52 +0100
Subject: [PATCH] [Flang][OpenMP] Move loop privatization out of dispatch
This pa
llvmbot wrote:
@llvm/pr-subscribers-compiler-rt-sanitizer
Author: None (llvmbot)
Changes
Backport 5ea9dd8c7076270695a1d90b9c73718e7d95e0bf
Requested by: @mstorsjo
---
Full diff: https://github.com/llvm/llvm-project/pull/106518.diff
3 Files Affected:
- (modified) compiler-rt/cmake/Modu
llvmbot wrote:
@vitalybuka What do you think about merging this PR to the release branch?
https://github.com/llvm/llvm-project/pull/106518
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
https://github.com/llvmbot created
https://github.com/llvm/llvm-project/pull/106518
Backport 5ea9dd8c7076270695a1d90b9c73718e7d95e0bf
Requested by: @mstorsjo
>From e5da34a3558d149d86b720135cb26f840f2b721f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Martin=20Storsj=C3=B6?=
Date: Thu, 8 Aug 2024 1
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/106518
___
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-clang
Author: cor3ntin (cor3ntin)
Changes
This reverts commit 2e1ad93961a3f444659c5d02d800e3144acccdb4.
Reverting #95202 in the 19.x branch
Fixes #106182
The change in #95202 causes code to crash and there is no good way to
backport a fix for that as
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/106516
___
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/cor3ntin created
https://github.com/llvm/llvm-project/pull/106516
This reverts commit 2e1ad93961a3f444659c5d02d800e3144acccdb4.
Reverting #95202 in the 19.x branch
Fixes #106182
The change in #95202 causes code to crash and there is no good way to backport
a fix for that a
https://github.com/cor3ntin milestoned
https://github.com/llvm/llvm-project/pull/106516
___
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
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (llvmbot)
Changes
Backport 47615ff2347a8be429404285de3b1c03b411e7af
Requested by: @ChuanqiXu9
---
Full diff: https://github.com/llvm/llvm-project/pull/106504.diff
3 Files Affected:
- (modified) clang/include/clang/Serialization/AS
https://github.com/llvmbot milestoned
https://github.com/llvm/llvm-project/pull/106504
___
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/106504
Backport 47615ff2347a8be429404285de3b1c03b411e7af
Requested by: @ChuanqiXu9
>From ec7bd37889f110c01a2445d1086c82c0e65d89a7 Mon Sep 17 00:00:00 2001
From: Chuanqi Xu
Date: Thu, 29 Aug 2024 15:42:57 +0800
Subjec
77 matches
Mail list logo