https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/75798
The attached test case would cause an assertion failure in Pointer.h when
operating on a null pointer.
>From 3367734775e1624521c5e0cab41072f8ba7a53ca Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
The attached test case would cause an assertion failure in Pointer.h when
operating on a null pointer.
---
Full diff: https://github.com/llvm/llvm-project/pull/75798.diff
3 Files Affected:
- (modified) cl
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/75485
>From e624652182dc0d76712f83a1c876ec856a8f38ad Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 14 Dec 2023 16:50:22 +0100
Subject: [PATCH] [clang][Interp] Support __real/__imag on primiti
RoboTux wrote:
> > > Not an expert on atomics, but why would we have a libcall for -O0 but not
> > > for O1 in the tests?
> >
> >
> > I looked at it for the u?(min|max) and it seemed to boil down to the atomic
> > expand pass being run at -O1 and above.
>
> No sorry, it's not that it's only
Fznamznon wrote:
@mikaelholmen , I see. I'll revert both patches to unblock you and I'll think
about the proper solution after the holidays.
Perhaps there should be wider discussion whether we should emulate GCC behavior
regarding silencing of these warnings.
IMO, the warnings you're seeing are
nico wrote:
Looks like the new test might be failing on macOS/arm64:
http://45.33.8.238/macm1/75171/step_11.txt
Maybe it just needs something like 806761a7629df268c8aed49657aeccffa6bca449,
but maybe the assert isn't benign.
Do you want to switch the test to use `-mtriple`, or do you want to r
mikaelholmen wrote:
@Fznamznon : Thank you!
I wider discussion about how clang should behave sounds good.
https://github.com/llvm/llvm-project/pull/70829
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
https://github.com/vhscampos edited
https://github.com/llvm/llvm-project/pull/75440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -752,10 +752,57 @@ __arm_st64bv0(void *__addr, data512_t __value) {
#define __arm_mte_ptrdiff(__ptra, __ptrb) __builtin_arm_subp(__ptra, __ptrb)
/* Memory Operations Intrinsics */
-#define __arm_mops_memset_tag(__tagged_address, __value, __size)\
+#define __arm_mops_mem
https://github.com/vhscampos requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/75440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> Looks like the new test might be failing on macOS/arm64:
> http://45.33.8.238/macm1/75171/step_11.txt
> Do you want to switch the test to use `-mtriple`, or do you want to revert
> and investigate what's up with the assert?
Really it needs #75469 for the triple to not default
vhscampos wrote:
There are some typos in the commit message and in the PR title.
https://github.com/llvm/llvm-project/pull/75440
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SamTebbs33 approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/75584
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> // At -O0, fast-regalloc cannot cope with the live vregs necessary to
This sounds extremely unsound
https://github.com/llvm/llvm-project/pull/74588
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
https://github.com/vpykhtin updated
https://github.com/llvm/llvm-project/pull/71556
>From 871f5eca90d9b3699faeef1b7847c6ea48981910 Mon Sep 17 00:00:00 2001
From: Valery Pykhtin
Date: Mon, 20 Nov 2023 15:22:16 +0100
Subject: [PATCH] add instcombine rule
---
clang/test/CodeGenOpenCL/builtins-am
https://github.com/doru1004 closed
https://github.com/llvm/llvm-project/pull/75642
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Gheorghe-Teodor Bercea
Date: 2023-12-18T09:47:59-05:00
New Revision: 4ef6587715bec4520332528c4a71fe5a9ac10477
URL:
https://github.com/llvm/llvm-project/commit/4ef6587715bec4520332528c4a71fe5a9ac10477
DIFF:
https://github.com/llvm/llvm-project/commit/4ef6587715bec4520332528c4a71fe5a9ac10
https://github.com/doru1004 closed
https://github.com/llvm/llvm-project/pull/72410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/BeMg created https://github.com/llvm/llvm-project/pull/75804
Address question from https://github.com/llvm/llvm-project/pull/74889
---
The target-cpu will infer its target-feature during backend codegen, it will
make TargetAttr doesn't work.
This patch implement the followi
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Piyou Chen (BeMg)
Changes
Address question from https://github.com/llvm/llvm-project/pull/74889
---
The target-cpu will infer its target-feature during backend codegen, it will
make TargetAttr doesn't work.
This patch implement
@@ -125,21 +126,28 @@ const std::string &CIndexer::getClangResourcesPath() {
#elif defined(_AIX)
getClangResourcesPathImplAIX(LibClangPath);
#else
+ bool pathFound = false;
fanbo-meng wrote:
nit: Variables should start with upper case
https://github.com/ll
https://github.com/fanbo-meng edited
https://github.com/llvm/llvm-project/pull/75637
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fanbo-meng approved this pull request.
LGTM with nit
https://github.com/llvm/llvm-project/pull/75637
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/abhina-sree updated
https://github.com/llvm/llvm-project/pull/75637
>From 61385bf2fa5c7369812eed6d40d04599f2383ee0 Mon Sep 17 00:00:00 2001
From: Abhina Sreeskantharajan
Date: Fri, 15 Dec 2023 14:39:23 -0500
Subject: [PATCH 1/2] check if we have dlfcn.h and dladdr
---
clang
@@ -125,21 +126,28 @@ const std::string &CIndexer::getClangResourcesPath() {
#elif defined(_AIX)
getClangResourcesPathImplAIX(LibClangPath);
#else
+ bool pathFound = false;
abhina-sree wrote:
thanks, done!
https://github.com/llvm/llvm-project/pull/75637
__
https://github.com/xu-chiheng edited
https://github.com/llvm/llvm-project/pull/74982
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xu-chiheng edited
https://github.com/llvm/llvm-project/pull/74982
___
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
Author: Sam Tebbs (SamTebbs33)
Changes
This patch adds a warning that's emitted when a builtin call uses ZA state but
the calling function doesn't provide any.
Patch by David Sherwood .
---
Patch is 186.65 KiB, truncated
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 d777504355dac9a2a8a7bbba6001d05b936868b2
a1840e028ef8b412ffb13b9d105e34fee54dfaed --
https://github.com/vpykhtin edited
https://github.com/llvm/llvm-project/pull/71556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nico wrote:
That did the trick, thanks!
https://github.com/llvm/llvm-project/pull/75475
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vpykhtin edited
https://github.com/llvm/llvm-project/pull/71556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jplehr wrote:
It appears that this patch made the buildbot unhappy
(https://lab.llvm.org/buildbot/#/builders/193/builds/43948).
Let me know if you need help with this.
https://github.com/llvm/llvm-project/pull/75642
___
cfe-commits mailing list
cfe-c
Author: Podchishchaeva, Mariya
Date: 2023-12-18T07:08:27-08:00
New Revision: 5cda366221236a43fdd89bca59e153b4384eaba8
URL:
https://github.com/llvm/llvm-project/commit/5cda366221236a43fdd89bca59e153b4384eaba8
DIFF:
https://github.com/llvm/llvm-project/commit/5cda366221236a43fdd89bca59e153b4384ea
https://github.com/momchil-velikov updated
https://github.com/llvm/llvm-project/pull/75117
>From 979b240d2a084eb87db43d3fabfffa8d3351d294 Mon Sep 17 00:00:00 2001
From: Momchil Velikov
Date: Mon, 11 Dec 2023 23:25:07 +
Subject: [PATCH 1/2] [Clang][SVE2.1] Add floating-point variants of
`sv
nico wrote:
Is `intrin0.h` a header that ships with MSVC's compiler, or with MS's STL? If
the latter, shouldn't everything just work already?
https://github.com/llvm/llvm-project/pull/75711
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
Author: Shilei Tian
Date: 2023-12-18T10:29:26-05:00
New Revision: 0f5eef1fb3e3f6491fb27dfb69b230946ffea69c
URL:
https://github.com/llvm/llvm-project/commit/0f5eef1fb3e3f6491fb27dfb69b230946ffea69c
DIFF:
https://github.com/llvm/llvm-project/commit/0f5eef1fb3e3f6491fb27dfb69b230946ffea69c.diff
L
https://github.com/shiltian closed
https://github.com/llvm/llvm-project/pull/68373
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Abhina Sree
Date: 2023-12-18T10:33:23-05:00
New Revision: 8a233d8cfde4cd026b5dd12b56ea029749a02329
URL:
https://github.com/llvm/llvm-project/commit/8a233d8cfde4cd026b5dd12b56ea029749a02329
DIFF:
https://github.com/llvm/llvm-project/commit/8a233d8cfde4cd026b5dd12b56ea029749a02329.diff
L
https://github.com/abhina-sree closed
https://github.com/llvm/llvm-project/pull/75637
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nico wrote:
Looks like it belongs to MSVC:
https://github.com/microsoft/STL/blob/a806c6960f1687590ffd4244794c753aa819/stl/inc/yvals_core.h#L2040
So I think the right order here is:
1. Add intrin0.h header
2. Wait for a release
3. Change MSSTL to include that in clang builds as well
4. Wait
nico wrote:
In other words, please undo the changes to `clang/lib/Headers/immintrin.h` for
now.
https://github.com/llvm/llvm-project/pull/75711
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
mizvekov wrote:
@kadircet ping
https://github.com/llvm/llvm-project/pull/75128
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sethp wrote:
Thanks for the feedback! I've definitely found it helpful trying to puzzle out
why one of my static_asserts is failing, though it's certainly a far less
complete solution than the document @tahonermann shared.
The output from `printPretty` will definitely fall apart for a complex
https://github.com/mjklemm created
https://github.com/llvm/llvm-project/pull/75816
At present, when building static or shared libraries, Flang adds
`-lFortran_main.a` (or `/WHOLEARCHIVE:Fortran.*.lib` pon Windows) to the link
line. This leads to the problem that `_QQmain` and `_QQEnvironmentD
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-clang
Author: Michael Klemm (mjklemm)
Changes
At present, when building static or shared libraries, Flang adds
`-lFortran_main.a` (or `/WHOLEARCHIVE:Fortran.*.lib` pon Windows) to the link
line. This leads to the prob
https://github.com/teresajohnson approved this pull request.
https://github.com/llvm/llvm-project/pull/75726
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/momchil-velikov updated
https://github.com/llvm/llvm-project/pull/75200
>From d97312680eff280210f588ef22416f845d31d2ef Mon Sep 17 00:00:00 2001
From: Momchil Velikov
Date: Tue, 12 Dec 2023 15:08:33 +
Subject: [PATCH 1/4] [Clang][SVE2.1] Make the part of the name optional
Author: Momchil Velikov
Date: 2023-12-18T15:52:28Z
New Revision: fd527def7edef8f8754b9e7812ed77f486bb144f
URL:
https://github.com/llvm/llvm-project/commit/fd527def7edef8f8754b9e7812ed77f486bb144f
DIFF:
https://github.com/llvm/llvm-project/commit/fd527def7edef8f8754b9e7812ed77f486bb144f.diff
LO
https://github.com/momchil-velikov closed
https://github.com/llvm/llvm-project/pull/75117
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/python3kgae approved this pull request.
https://github.com/llvm/llvm-project/pull/75674
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kmclaughlin-arm created
https://github.com/llvm/llvm-project/pull/75821
This patch enables the following builtins for SME2:
- svld1, svld1_vnum
- svldnt1, svldnt1_vnum
- svst1, svst1_vnum
- svstnt1, svstnt1_vnum
>From 1b2022f34ad3b038f714d8d0559f4e129d5e731a Mon Sep 17 00
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-clang
Author: Kerry McLaughlin (kmclaughlin-arm)
Changes
This patch enables the following builtins for SME2:
- svld1, svld1_vnum
- svldnt1, svldnt1_vnum
- svst1, svst1_vnum
- svstnt1, svstnt1_vnum
---
Full diff: ht
https://github.com/SamTebbs33 updated
https://github.com/llvm/llvm-project/pull/71927
>From 93a02d9af1d7e4f1e23c252d72b20d292927a79f Mon Sep 17 00:00:00 2001
From: Samuel Tebbs
Date: Thu, 9 Nov 2023 17:39:26 +
Subject: [PATCH 1/6] [AArch64][SME2] Enable bfm builtings for sme2
This patch en
https://github.com/sdesmalen-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/75805
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
SamTebbs33 wrote:
Should these builtins be `IsStreamingCompatible` until we add
`IsStreamingOrSVE2p1`?
https://github.com/llvm/llvm-project/pull/75821
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/SamTebbs33 closed
https://github.com/llvm/llvm-project/pull/75805
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdesmalen-arm approved this pull request.
LGTM with comment addressed.
https://github.com/llvm/llvm-project/pull/71927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
@@ -2,13 +2,24 @@
// REQUIRES: aarch64-registered-target
// RUN: %clang_cc1 -fclang-abi-compat=latest -triple aarch64-none-linux-gnu
-target-feature +sve2p1 -S -disable-O0-optnone -Werror -Wall -emit-llvm -o - %s
| opt -S -p mem2reg,instcombine,tailcallelim | FileCheck %s
+/
https://github.com/sdesmalen-arm edited
https://github.com/llvm/llvm-project/pull/71927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -11881,6 +11896,8 @@ CheckPrintfHandler::checkFormatExpr(const
analyze_printf::PrintfSpecifier &FS,
ICE->getType() == S.Context.UnsignedIntTy) {
// All further checking is done on the subexpression
ImplicitMatch = AT.matchesType(S.Context, ExprTy);
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/75635
>From 336d1629f38a8681038f026c599a256761b522dc Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Fri, 15 Dec 2023 11:13:30 -0700
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
https://github.com/sdesmalen-arm updated
https://github.com/llvm/llvm-project/pull/75791
>From 8b30f772982d7d608541526b1fa42de2b5eb7e1e Mon Sep 17 00:00:00 2001
From: Sander de Smalen
Date: Mon, 18 Dec 2023 12:55:30 +
Subject: [PATCH 1/2] [Clang][AArch64] Add missing SME functions to header
https://github.com/SamTebbs33 updated
https://github.com/llvm/llvm-project/pull/71927
>From 93a02d9af1d7e4f1e23c252d72b20d292927a79f Mon Sep 17 00:00:00 2001
From: Samuel Tebbs
Date: Thu, 9 Nov 2023 17:39:26 +
Subject: [PATCH 1/7] [AArch64][SME2] Enable bfm builtings for sme2
This patch en
@@ -434,8 +429,21 @@ FailureOr
linalg::lowerUnPack(RewriterBase &rewriter,
RankedTensorType::Builder(packedTensorType).setShape(stripMinedShape);
RankedTensorType collapsedType = tensor::CollapseShapeOp::inferCollapsedType(
stripMinedTensorType, packingMetadata.r
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/75635
>From 336d1629f38a8681038f026c599a256761b522dc Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Fri, 15 Dec 2023 11:13:30 -0700
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
https://github.com/jroelofs updated
https://github.com/llvm/llvm-project/pull/75635
>From 336d1629f38a8681038f026c599a256761b522dc Mon Sep 17 00:00:00 2001
From: Jon Roelofs
Date: Fri, 15 Dec 2023 11:13:30 -0700
Subject: [PATCH 1/4] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
https://github.com/ldionne updated
https://github.com/llvm/llvm-project/pull/75646
>From c5ff17c24afba4b44e45894452c733fef0183fb0 Mon Sep 17 00:00:00 2001
From: Louis Dionne
Date: Fri, 15 Dec 2023 14:14:44 -0500
Subject: [PATCH 1/2] [libunwind] Use -nostdlib++ when linking libunwind
We shouldn
bd1976bris wrote:
Noting previous discussions w.r.t. -fvisibility-global-new-delete-hidden on
https://reviews.llvm.org/D133266. I have added reviewers from that phab review
in case they are interested in this.
https://github.com/llvm/llvm-project/pull/75364
https://github.com/kmclaughlin-arm closed
https://github.com/llvm/llvm-project/pull/74841
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/sdesmalen-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/71927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2091,6 +2091,12 @@ bool CompilerInvocation::ParseCodeGenArgs(CodeGenOptions
&Opts, ArgList &Args,
bool UsingProfile =
UsingSampleProfile || !Opts.ProfileInstrumentUsePath.empty();
+ if (Args.hasArg(options::OPT_fdiagnostics_show_profile_count) &&
https://github.com/momchil-velikov edited
https://github.com/llvm/llvm-project/pull/75200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/momchil-velikov edited
https://github.com/llvm/llvm-project/pull/75200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/momchil-velikov edited
https://github.com/llvm/llvm-project/pull/75200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Momchil Velikov
Date: 2023-12-18T16:49:59Z
New Revision: 35a77fc13ce62f4bc21b65c130801cb5f42fb228
URL:
https://github.com/llvm/llvm-project/commit/35a77fc13ce62f4bc21b65c130801cb5f42fb228
DIFF:
https://github.com/llvm/llvm-project/commit/35a77fc13ce62f4bc21b65c130801cb5f42fb228.diff
LO
https://github.com/momchil-velikov closed
https://github.com/llvm/llvm-project/pull/75200
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Sam Tebbs
Date: 2023-12-18T16:51:10Z
New Revision: fade67565e1a3d7447335f9b5dfbf85dccb5773b
URL:
https://github.com/llvm/llvm-project/commit/fade67565e1a3d7447335f9b5dfbf85dccb5773b
DIFF:
https://github.com/llvm/llvm-project/commit/fade67565e1a3d7447335f9b5dfbf85dccb5773b.diff
LOG: [AA
https://github.com/SamTebbs33 closed
https://github.com/llvm/llvm-project/pull/71927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/googlewalt created
https://github.com/llvm/llvm-project/pull/75827
…ules
We use this to verify properties of our build graph, even when modules are off.
Xcode 14.3.1 seems to have dropped these flags so we are creating a unit test
to reproduce the issue.
>From 0e83c457b51
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Walter Lee (googlewalt)
Changes
…ules
We use this to verify properties of our build graph, even when modules are off.
Xcode 14.3.1 seems to have dropped these flags so we are creating a unit test
to reproduce the issue.
---
Full
https://github.com/srcarroll updated
https://github.com/llvm/llvm-project/pull/75494
>From 3879e633f4e4ba36ca24e6a870942b695ac935e0 Mon Sep 17 00:00:00 2001
From: Sam
Date: Wed, 13 Dec 2023 15:25:31 -0600
Subject: [PATCH 1/7] [MLIR][Linalg] Support dynamic sizes in `lower_unpack`
---
.../Dial
https://github.com/minglotus-6 closed
https://github.com/llvm/llvm-project/pull/74008
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Joseph Huber
Date: 2023-12-18T11:16:49-06:00
New Revision: 8e2cc19e2e60d348859c0967a2707bcc8cb9936a
URL:
https://github.com/llvm/llvm-project/commit/8e2cc19e2e60d348859c0967a2707bcc8cb9936a
DIFF:
https://github.com/llvm/llvm-project/commit/8e2cc19e2e60d348859c0967a2707bcc8cb9936a.diff
https://github.com/ye-luo approved this pull request.
https://github.com/llvm/llvm-project/pull/75757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ye-luo closed https://github.com/llvm/llvm-project/pull/75757
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -362,14 +362,20 @@ mlir::scf::tileUsingSCFForOp(RewriterBase &rewriter,
TilingInterface op,
auto clonedOp = cast(
cloneOpAndUpdateDestinationArgs(rewriter, op, clonedOpDestination));
- // 5b. Tile the cloned operation.
+ // 5b. Early return cloned op if tiling is
@@ -362,14 +362,20 @@ mlir::scf::tileUsingSCFForOp(RewriterBase &rewriter,
TilingInterface op,
auto clonedOp = cast(
cloneOpAndUpdateDestinationArgs(rewriter, op, clonedOpDestination));
- // 5b. Tile the cloned operation.
+ // 5b. Early return cloned op if tiling is
@@ -434,8 +428,14 @@ FailureOr
linalg::lowerUnPack(RewriterBase &rewriter,
RankedTensorType::Builder(packedTensorType).setShape(stripMinedShape);
RankedTensorType collapsedType = tensor::CollapseShapeOp::inferCollapsedType(
stripMinedTensorType, packingMetadata.r
https://github.com/chelini edited
https://github.com/llvm/llvm-project/pull/75494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -464,6 +464,129 @@ module attributes {transform.with_named_sequence} {
// -
+// Check that we can lower unpack with dynamic dimensions in the input and
destination.
+// CHECK-LABEL: func.func @unpack_with_dynamic_input_dest(
+// CHECK-SAME: %[[ARG0:.*]]: tensor, %[[AR
https://github.com/chelini approved this pull request.
Thanks, it looks good to me.
https://github.com/llvm/llvm-project/pull/75494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -253,35 +246,6 @@ void UAVResource::print(raw_ostream &OS) const {
ResourceBase::print(OS, "U", "u");
}
-// FIXME: Capture this in HLSL source. I would go do this right now, but I want
-// to get this in first so that I can make sure to capture all the extra
-// informati
https://github.com/srcarroll updated
https://github.com/llvm/llvm-project/pull/75494
>From 3879e633f4e4ba36ca24e6a870942b695ac935e0 Mon Sep 17 00:00:00 2001
From: Sam
Date: Wed, 13 Dec 2023 15:25:31 -0600
Subject: [PATCH 1/8] [MLIR][Linalg] Support dynamic sizes in `lower_unpack`
---
.../Dial
@@ -216,15 +214,62 @@ void
CGHLSLRuntime::addBufferResourceAnnotation(llvm::GlobalVariable *GV,
assert(false && "Unsupported buffer type!");
return;
}
-
assert(ResourceMD != nullptr &&
"ResourceMD must have been set by the switch above.");
llvm::hls
ributzka wrote:
> This looks pretty nice, I left just a couple of notes. You might want to take
> a look at the CI failures.
Thanks Jan, I will take a look. At first glance I need to build and fix `flang`
too.
https://github.com/llvm/llvm-project/pull/75530
___
Author: Fangrui Song
Date: 2023-12-18T09:46:58-08:00
New Revision: 96aca7c51701f9b3c5dd8567fcddf29492008e6d
URL:
https://github.com/llvm/llvm-project/commit/96aca7c51701f9b3c5dd8567fcddf29492008e6d
DIFF:
https://github.com/llvm/llvm-project/commit/96aca7c51701f9b3c5dd8567fcddf29492008e6d.diff
https://github.com/MaskRay closed
https://github.com/llvm/llvm-project/pull/75726
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
shafik wrote:
Can you please update your summary to explain the problem and how the fix
addresses the problem.
This is usually what goes into the git log and we want those to be as
descriptive as possible but also it help code reviewers to understand what they
are reviewing quicker.
https://
@@ -224,19 +233,20 @@ class CompilerInvocation : public CompilerInvocationBase {
/// @{
// Note: These need to be pulled in manually. Otherwise, they get hidden by
// the mutable getters with the same names.
- using CompilerInvocationBase::getLangOpts;
- using CompilerI
101 - 200 of 419 matches
Mail list logo