Author: Vitaly Buka
Date: 2024-08-16T23:02:12-07:00
New Revision: d257cd875873666d250e3fa7e95aafcd099f4e2f
URL:
https://github.com/llvm/llvm-project/commit/d257cd875873666d250e3fa7e95aafcd099f4e2f
DIFF:
https://github.com/llvm/llvm-project/commit/d257cd875873666d250e3fa7e95aafcd099f4e2f.diff
L
@@ -45,7 +45,7 @@ void test3(packedfloat3 *p) {
*p = (packedfloat3) { 3.2f, 2.3f, 0.1f };
}
// CHECK: @test3(
-// CHECK: store <4 x float> {{.*}}, align 4
+// CHECK: store <3 x float> {{.*}}, align 4
arsenm wrote:
According to the langref the backend can wid
https://github.com/efriedma-quic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/101738
___
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: None (c8ef)
Changes
fix: #55390.
---
Full diff: https://github.com/llvm/llvm-project/pull/104666.diff
2 Files Affected:
- (modified) clang/lib/AST/ExprConstant.cpp (+4-3)
- (added) clang/test/SemaCXX/complex-div.cpp (+3)
``d
https://github.com/c8ef created https://github.com/llvm/llvm-project/pull/104666
fix: #55390.
>From b58b9c3ad5fb2b37715ba9f52c905b6961159f0c Mon Sep 17 00:00:00 2001
From: c8ef
Date: Sat, 17 Aug 2024 05:42:39 +
Subject: [PATCH] [clang] fix divide by zero in ComplexExprEvaluator
---
clang/
banach-space wrote:
Hi Kareem, thanks for the fix! Is there any reference that would document .i
files? Perhaps GFortran docs? This might be sth standard, but we should capture
that in our docs nonetheless (eg in a commit summary).
Btw, I will be OOO for a while, so apologies in advance if th
https://github.com/ergawy updated
https://github.com/llvm/llvm-project/pull/104664
>From 1f504bf784ee3b19ed29d2db1ba4ba26ac7d7d66 Mon Sep 17 00:00:00 2001
From: ergawy
Date: Sat, 17 Aug 2024 00:20:11 -0500
Subject: [PATCH] [flang][driver] Add pre-processing type to `.i` files
This diff allows
llvmbot wrote:
@llvm/pr-subscribers-flang-driver
Author: Kareem Ergawy (ergawy)
Changes
This diff allows `.i` files emitted by flang-new to be treated as valid files
in the pre-processing phase. This, in turn, allows flang-new to add
pre-processing options (e.g. `-I`) when launching comp
https://github.com/ergawy created
https://github.com/llvm/llvm-project/pull/104664
This diff allows `.i` files emitted by flang-new to be treated as valid files
in the pre-processing phase. This, in turn, allows flang-new to add
pre-processing options (e.g. `-I`) when launching compilation job
@@ -45,7 +45,7 @@ void test3(packedfloat3 *p) {
*p = (packedfloat3) { 3.2f, 2.3f, 0.1f };
}
// CHECK: @test3(
-// CHECK: store <4 x float> {{.*}}, align 4
+// CHECK: store <3 x float> {{.*}}, align 4
topperc wrote:
Is the backend able to lower this a a singl
@@ -152,12 +152,22 @@ namespace clang {
};
}
+ namespace RISCVXCV {
+ enum {
+LastRVVBuiltin = RISCVVector::FirstTSBuiltin - 1,
+#define BUILTIN(ID, TYPE, ATTRS) BI__builtin_riscv_cv_##ID,
+#include "clang/Basic/BuiltinsRISCVXCV.def"
topperc wrote:
owenca wrote:
There's a compiler warning:
```
In file included from
/Users/Owen/llvm-project/clang/lib/Format/AffectedRangeManager.cpp:16:
/Users/Owen/llvm-project/clang/lib/Format/FormatToken.h:1982:13: warning:
unused function 'startsNextParameter' [-Wunused-function]
1982 | static bool star
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/104663.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+11)
- (modified) clang/test/AST/ByteCode/cxx23.cpp (+14)
`
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/104663
None
>From 3faff3df9ebadbc12e34d15c96742856020bf7f1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Sat, 17 Aug 2024 06:15:45 +0200
Subject: [PATCH] [clang][bytecode] Fix shifting negative v
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
This happens for enum types with bool parent types. isBooleanType() returns
false for them however.
The previous version did the same thing by re-classifying the enum integer
type, but that breaks with forw
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/104662
This happens for enum types with bool parent types. isBooleanType() returns
false for them however.
The previous version did the same thing by re-classifying the enum integer
type, but that breaks with forwar
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-backend-amdgpu
Author: Shilei Tian (shiltian)
Changes
Clang uses a long-time special handling of the case where 3 element vector
loads
and stores are performed as 4 element, and then a shufflevector is used to
extrac
shiltian wrote:
* **#104661** https://app.graphite.dev/github/pr/llvm/llvm-project/104661?utm_source=stack-comment-icon";
target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite"
width="10px" height="10px"/> 👈
* `main`
This stack of pull requests is managed by Grap
https://github.com/shiltian ready_for_review
https://github.com/llvm/llvm-project/pull/104661
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shiltian created
https://github.com/llvm/llvm-project/pull/104661
Clang uses a long-time special handling of the case where 3 element vector
loads
and stores are performed as 4 element, and then a shufflevector is used to
extract
the used elements. Odd sized vector codegen
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-x86_64-linux-fast` running on `sanitizer-buildbot3` while building
`clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/169/builds/2242
Here is the relevant piece of
https://github.com/pow2clk updated
https://github.com/llvm/llvm-project/pull/104656
>From 9aff63478b76f042c05b7ae3dd1a2c099dc615de Mon Sep 17 00:00:00 2001
From: Greg Roth
Date: Fri, 2 Aug 2024 20:10:04 -0600
Subject: [PATCH 1/2] Add SPIRV generation for HLSL dot
This adds the SPIRV fdot, sdot
pow2clk wrote:
Here's the new PR for anyone who wants to keep following along in its altered
state: #104656
https://github.com/llvm/llvm-project/pull/102872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
https://github.com/urnathan closed
https://github.com/llvm/llvm-project/pull/102078
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nathan Sidwell
Date: 2024-08-16T21:05:08-04:00
New Revision: 4cf9a4266167c686e7e405ead5d6c1cd389eeca5
URL:
https://github.com/llvm/llvm-project/commit/4cf9a4266167c686e7e405ead5d6c1cd389eeca5
DIFF:
https://github.com/llvm/llvm-project/commit/4cf9a4266167c686e7e405ead5d6c1cd389eeca5.diff
llvmbot wrote:
@llvm/pr-subscribers-llvm-ir
Author: Greg Roth (pow2clk)
Changes
This adds the SPIRV fdot, sdot, and udot intrinsics and allows them to be
created at codegen depending on the target architecture. This required moving
some of the DXIL-specific choices to DXIL instruction ex
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-backend-spir-v
@llvm/pr-subscribers-backend-directx
Author: Greg Roth (pow2clk)
Changes
This adds the SPIRV fdot, sdot, and udot intrinsics and allows them to be
created at codegen depending on the target architecture.
https://github.com/pow2clk edited
https://github.com/llvm/llvm-project/pull/104656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/pow2clk created
https://github.com/llvm/llvm-project/pull/104656
This adds the SPIRV fdot, sdot, and udot intrinsics and allows them to be
created at codegen depending on the target architecture. This required moving
some of the DXIL-specific choices to DXIL instruction expa
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Ian Anderson (ian-twilightcoder)
Changes
Add the supported application extension and Mac Catalyst platforms to the
availability attribute documentation.
---
Full diff: https://github.com/llvm/llvm-project/pull/104653.diff
1 Files Affect
https://github.com/ian-twilightcoder created
https://github.com/llvm/llvm-project/pull/104653
Add the supported application extension and Mac Catalyst platforms to the
availability attribute documentation.
>From f833ec7e5e1dd8d8076df157ab0766b87ea0734b Mon Sep 17 00:00:00 2001
From: Ian Anders
pow2clk wrote:
Closing in light of the above. A new PR will capture the DXIL and
SPIRV-specific work
https://github.com/llvm/llvm-project/pull/102872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
https://github.com/pow2clk closed
https://github.com/llvm/llvm-project/pull/102872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ahmedbougacha edited
https://github.com/llvm/llvm-project/pull/104435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/104460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Joseph Huber
Date: 2024-08-16T18:36:27-05:00
New Revision: e169cc162adbe89d498e774bccf4e228af989849
URL:
https://github.com/llvm/llvm-project/commit/e169cc162adbe89d498e774bccf4e228af989849
DIFF:
https://github.com/llvm/llvm-project/commit/e169cc162adbe89d498e774bccf4e228af989849.diff
https://github.com/bogner commented:
If I understand correctly from reading this, there should never be a case where
both dxc and clang both choose an overload but those differ. Ie, any case where
the behaviour disagrees will result in clang failing to resolve the overload
because it's ambiguo
https://github.com/dougsonos edited
https://github.com/llvm/llvm-project/pull/99656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,256 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -std=c++20 -verify
%s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the FXAnalysis pass from running at all.
+
+// This diagnostic is re-enabled a
@@ -0,0 +1,90 @@
+// Use --implicit-check-not to ensure no additional CPUs are in this list
+
+// RUN: not %clang_cc1 -triple arm64--- -target-cpu not-a-cpu -fsyntax-only %s
2>&1 | FileCheck %s --implicit-check-not={{[a-zA-Z0-9]}}
+// RUN: not %clang_cc1 -triple arm64--- -tune-cp
https://github.com/Artem-B approved this pull request.
https://github.com/llvm/llvm-project/pull/104460
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/QuietMisdreavus approved this pull request.
https://github.com/llvm/llvm-project/pull/104600
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -892,7 +905,7 @@ def ProcessorFeatures {
FeatureComplxNum, FeatureCRC, FeatureJS,
FeatureLSE, FeaturePAuth, FeatureFPAC,
FeatureRAS, FeatureRCPC, FeatureRDM,
-
@@ -845,7 +857,8 @@ def ProcessorFeatures {
list AppleA12 = [HasV8_3aOps, FeatureSHA2, FeatureAES,
FeatureFPARMv8,
FeatureNEON, FeaturePerfMon,
FeatureFullFP16,
FeatureComplxNum, FeatureCRC, FeatureJS
https://github.com/ahmedbougacha commented:
A couple small issues inline, LGTM otherwise, thanks!
>> FEAT_SSBS is not mandatory for any architecture.
>
> https://reviews.llvm.org/D54629 says it is mandatory for 8.5-a but I can't
> see that in the Arm ARM.
It was indeed originally mandatory (se
https://github.com/ahmedbougacha edited
https://github.com/llvm/llvm-project/pull/104435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: PeterChou1
Date: 2024-08-16T18:37:17-04:00
New Revision: 5ef2456a438578b0783241a2744efc62d47e5ab6
URL:
https://github.com/llvm/llvm-project/commit/5ef2456a438578b0783241a2744efc62d47e5ab6
DIFF:
https://github.com/llvm/llvm-project/commit/5ef2456a438578b0783241a2744efc62d47e5ab6.diff
LO
https://github.com/PeterChou1 closed
https://github.com/llvm/llvm-project/pull/101282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/104594
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Oleksandr T.
Date: 2024-08-17T00:01:33+02:00
New Revision: e4f3735d5f600b17b8f86956162d41ce82096685
URL:
https://github.com/llvm/llvm-project/commit/e4f3735d5f600b17b8f86956162d41ce82096685
DIFF:
https://github.com/llvm/llvm-project/commit/e4f3735d5f600b17b8f86956162d41ce82096685.diff
bogner wrote:
> There is some discussion in the RFC, but I don't see a consensus on the "dot"
> intrinsic in particular. I personally haven't found the arguments in favor of
> it particularly compelling.
>
> This really needs an RFC specific to that intrinsic (class), which includes a
> clear
@@ -76,8 +79,75 @@ class HIPUndefinedFatBinSymbols {
return GPUBinHandleSymbols;
}
+ // Collect symbols from static libraries specified by -l options.
+ void processStaticLibraries() {
+llvm::SmallVector LibNames;
+llvm::SmallVector LibPaths;
+llvm::SmallVe
@@ -76,8 +79,75 @@ class HIPUndefinedFatBinSymbols {
return GPUBinHandleSymbols;
}
+ // Collect symbols from static libraries specified by -l options.
+ void processStaticLibraries() {
+llvm::SmallVector LibNames;
+llvm::SmallVector LibPaths;
+llvm::SmallVe
https://github.com/Artem-B approved this pull request.
LGTM in general with a couple of nits.
https://github.com/llvm/llvm-project/pull/104638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
https://github.com/Artem-B edited
https://github.com/llvm/llvm-project/pull/104638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -402,6 +402,21 @@
// MCPU-SIFIVE-P670-SAME: "-target-feature" "+zvkt"
// MCPU-SIFIVE-P670-SAME: "-target-abi" "lp64d"
+// RUN: %clang -target riscv32 -### -c %s 2>&1 -mcpu=rp2350-hazard3 |
FileCheck -check-prefix=MCPU-HAZARD3 %s
+// MCPU-HAZARD3: "-target-cpu" "rp2350-haza
mstorsjo wrote:
> @mstorsjo I think you put the finger on something I don't really understand
> at the moment. What is the relationship between `armv7` and `thumbv7` as a
> target triple? Is there any documentation on what an "effective triple" is in
> comparison to a "non-effective triple"?
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-hip-vega20` running
on `hip-vega20-0` while building `clang` at step 3 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/123/builds/3819
Here is the relevant piece of the build log for
Author: Valentin Clement (バレンタイン クレメン)
Date: 2024-08-16T13:41:24-07:00
New Revision: 1a8817aebe12b8d4708ee849fb8f792cffb7ae85
URL:
https://github.com/llvm/llvm-project/commit/1a8817aebe12b8d4708ee849fb8f792cffb7ae85
DIFF:
https://github.com/llvm/llvm-project/commit/1a8817aebe12b8d4708ee849fb8f7
carlosgalvezp wrote:
> An inconsistency is that readability-uppercase-literal-suffix only handles l
> and u by default whereas the check here also handles f
Then that check should be updated to handle the F suffix as well.
It doesn't make sense to have different style for different suffixes:
Author: Valentin Clement
Date: 2024-08-16T13:19:26-07:00
New Revision: de9338fc625d8d67c18032492b2e743cd53f4b37
URL:
https://github.com/llvm/llvm-project/commit/de9338fc625d8d67c18032492b2e743cd53f4b37
DIFF:
https://github.com/llvm/llvm-project/commit/de9338fc625d8d67c18032492b2e743cd53f4b37.di
https://github.com/dougsonos edited
https://github.com/llvm/llvm-project/pull/99656
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,256 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -std=c++20 -verify
%s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the FXAnalysis pass from running at all.
+
+// This diagnostic is re-enabled a
vitalybuka wrote:
> > Can we split `-fsanitize=unsigned-integer-overflow` into
> > `-fsanitize=unsigned-integer-overflow-patternA,unsigned-integer-overflow-patternB,unsigned-integer-overflow-patternC...`
> > ?
> > Then it's quite intuitive to disable them with `no-sanitize`.
>
> Yikes, no way.
https://github.com/rjmccall approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/103044
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Valentin Clement (バレンタイン クレメン)
Date: 2024-08-16T13:10:36-07:00
New Revision: af81b4f9f98f6ac89f876637f065d6525e374468
URL:
https://github.com/llvm/llvm-project/commit/af81b4f9f98f6ac89f876637f065d6525e374468
DIFF:
https://github.com/llvm/llvm-project/commit/af81b4f9f98f6ac89f876637f065d
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-driver
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
For -fgpu-rdc linking, clang needs to collect undefined fatbin symbols and
resolve them to the embedded fatbin.
This has been done for object files and archive files pas
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/104638
For -fgpu-rdc linking, clang needs to collect undefined fatbin symbols and
resolve them to the embedded fatbin.
This has been done for object files and archive files passed as input files to
clang.
However,
@@ -0,0 +1,220 @@
+//===- AMDGPUMCResourceInfo.cpp --- MC Resource Info
--===//
+//
+// 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: Ap
@@ -0,0 +1,220 @@
+//===- AMDGPUMCResourceInfo.cpp --- MC Resource Info
--===//
+//
+// 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: Ap
@@ -0,0 +1,220 @@
+//===- AMDGPUMCResourceInfo.cpp --- MC Resource Info
--===//
+//
+// 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: Ap
@@ -0,0 +1,220 @@
+//===- AMDGPUMCResourceInfo.cpp --- MC Resource Info
--===//
+//
+// 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: Ap
https://github.com/slinder1 commented:
LGTM, with some nits
The expressions look pretty reasonable to me, although I didn't really go
through validating that they are correct
https://github.com/llvm/llvm-project/pull/102913
___
cfe-commits mailing li
https://github.com/slinder1 edited
https://github.com/llvm/llvm-project/pull/102913
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Da-Viper updated
https://github.com/llvm/llvm-project/pull/102831
>From 8a4f6af9fc1f44c2f8b5fd3693ca14eaf776fd02 Mon Sep 17 00:00:00 2001
From: Ebuka Ezike
Date: Sun, 11 Aug 2024 21:39:35 +0100
Subject: [PATCH 01/11] [clang-tidy] use upper cace letters for bool conversion
su
https://github.com/haoNoQ approved this pull request.
Works for me! IIUC nobody touched this code in a while, and that's probably
because it's basically perfect for everyone's existing purposes. So I think
moving it to ADT is appropriate.
https://github.com/llvm/llvm-project/pull/99770
___
lei137 wrote:
> I'm not an expert here, but I think the change you made makes sense since the
> `Ty` is supposed to be the first field.
>
> Also, might be a dumb question, I saw we have 32-bit Linux run lines, but is
> that still worth testing?
Thanks for taking a look @amy-kwan. The 32bit l
@@ -0,0 +1,256 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -std=c++20 -verify
%s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the FXAnalysis pass from running at all.
+
+// This diagnostic is re-enabled a
vitalybuka wrote:
> > How i can see the full amount of breakage on different platforms
> > proactively? I have created a PR: #104607 to address the big endian issue
> > with the test as my // REQUIRES: x86-registered-target turned out not to be
> > enough to exclude running test in the big end
@@ -0,0 +1,256 @@
+// RUN: %clang_cc1 -fsyntax-only -fblocks -fcxx-exceptions -std=c++20 -verify
%s
+// These are in a separate file because errors (e.g. incompatible attributes)
currently prevent
+// the FXAnalysis pass from running at all.
+
+// This diagnostic is re-enabled a
zeroomega wrote:
> I think those are failing because of unexpected passes in libc++ tests:
>
> ```
> TEST 'llvm-libc++-static-clangcl.cfg.in ::
> libcxx/utilities/expected/expected.void/transform_error.mandates.verify.cpp'
> FAILED
> Exit Code: 0
>
>
https://github.com/amy-kwan commented:
I'm not an expert here, but I think the change you made makes sense since the
`Ty` is supposed to be the first field.
Also, might be a dumb question, I saw we have 32-bit Linux run lines, but is
that still worth testing?
https://github.com/llvm/llvm-proj
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `ppc64le-flang-rhel-clang`
running on `ppc64le-flang-rhel-test` while building `clang,flang` at step 6
"test-build-unified-tree-check-flang".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/157/builds/5
@@ -4348,8 +4348,18 @@ LValue CodeGenFunction::EmitMatrixSubscriptExpr(const
MatrixSubscriptExpr *E) {
!E->isIncomplete() &&
"incomplete matrix subscript expressions should be rejected during
Sema");
LValue Base = EmitLValue(E->getBase());
- llvm::Value *RowIdx
@@ -1121,3 +1121,99 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
}
return false;
}
+
+bool SemaHLSL::CheckCompatibleParameterABI(FunctionDecl *New,
+ FunctionDecl *Old) {
+ if (New->getNumPar
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Valentin Clement (バレンタイン クレメン) (clementval)
Changes
Reverts llvm/llvm-project#104613
---
Full diff: https://github.com/llvm/llvm-project/pull/104632.diff
2 Files Affected:
- (modified) clang/lib/Driver/Types.cpp (-3)
- (modified
https://github.com/clementval closed
https://github.com/llvm/llvm-project/pull/104632
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Valentin Clement (バレンタイン クレメン)
Date: 2024-08-16T12:02:37-07:00
New Revision: e315ba185b835c8ee520076b434377cfb7350067
URL:
https://github.com/llvm/llvm-project/commit/e315ba185b835c8ee520076b434377cfb7350067
DIFF:
https://github.com/llvm/llvm-project/commit/e315ba185b835c8ee520076b43437
https://github.com/clementval created
https://github.com/llvm/llvm-project/pull/104632
Reverts llvm/llvm-project#104613
>From cf8d13e27b647afaca66457d93b45a7da0e9cdfb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Valentin=20Clement=20=28=E3=83=90=E3=83=AC=E3=83=B3?=
=?UTF-8?q?=E3=82=BF=E3=82=A4=E3=
@@ -108,6 +108,7 @@ Changes to the RISC-V Backend
fill value) rather than NOPs.
* Added Syntacore SCR4 and SCR5 CPUs: ``-mcpu=syntacore-scr4/5-rv32/64``
* ``-mcpu=sifive-p470`` was added.
+* Added Hazard3 CPU: ``-mcpu=hazard3`` (32-bit only).
lenary wrote:
D
@@ -402,6 +402,21 @@
// MCPU-SIFIVE-P670-SAME: "-target-feature" "+zvkt"
// MCPU-SIFIVE-P670-SAME: "-target-abi" "lp64d"
+// RUN: %clang -target riscv32 -### -c %s 2>&1 -mcpu=rp2350-hazard3 |
FileCheck -check-prefix=MCPU-HAZARD3 %s
+// MCPU-HAZARD3: "-target-cpu" "rp2350-haza
@@ -108,6 +108,7 @@ Changes to the RISC-V Backend
fill value) rather than NOPs.
* Added Syntacore SCR4 and SCR5 CPUs: ``-mcpu=syntacore-scr4/5-rv32/64``
* ``-mcpu=sifive-p470`` was added.
+* Added Hazard3 CPU: ``-mcpu=hazard3`` (32-bit only).
Wren6991 wrote:
nikic wrote:
> > Please create a separate RFC for these intrinsics. I don't think there is a
> > consensus on these intrinsics, and
> > https://discourse.llvm.org/t/rfc-all-the-math-intrinsics/78294 covers way
> > too much disparate ground (something like "tan" and something like "dot"
> > ar
@@ -5663,6 +5663,8 @@ def err_explicit_instantiation_internal_linkage : Error<
def err_explicit_instantiation_not_known : Error<
"explicit instantiation of %0 does not refer to a function template, "
"variable template, member function, member class, or static data member">
https://github.com/clementval closed
https://github.com/llvm/llvm-project/pull/104613
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Valentin Clement (バレンタイン クレメン)
Date: 2024-08-16T11:56:22-07:00
New Revision: e6b9f12b0ac0f1e6f7f7145719092c10731e4fe4
URL:
https://github.com/llvm/llvm-project/commit/e6b9f12b0ac0f1e6f7f7145719092c10731e4fe4
DIFF:
https://github.com/llvm/llvm-project/commit/e6b9f12b0ac0f1e6f7f7145719092
ichaer wrote:
@mydeveloperday, what do you think, is this good to merge?
https://github.com/llvm/llvm-project/pull/96804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1285,6 +1289,34 @@ static unsigned getBoolCmpOpcode(unsigned PredNum) {
}
}
+static APFloat getZeroFP(const Type *LLVMFloatTy) {
farzonl wrote:
I'm a little suprised this didn't already exist.
https://github.com/llvm/llvm-project/pull/104619
__
@@ -362,6 +364,34 @@ static bool expandClampIntrinsic(CallInst *Orig,
Intrinsic::ID ClampIntrinsic) {
return true;
}
+static bool expandSaturateIntrinsic(CallInst *SaturateCall) {
farzonl wrote:
wait for @pow2clk change merges then change this to return `
@@ -551,24 +541,21 @@ getHLSLResourceAttrFromEitherDecl(VarDecl *VD,
// the resource attr could be on the record decl itself or on one of
// its fields (the resource handle, most commonly)
bob80905 wrote:
Yes, I think I may have left support for that
1 - 100 of 395 matches
Mail list logo
