https://github.com/kpdev updated https://github.com/llvm/llvm-project/pull/78253
>From 9850957ff41dd5cbe72fd76925a32ad9c277d5e1 Mon Sep 17 00:00:00 2001
From: Nikolai Kholiavin
Date: Wed, 20 Dec 2023 16:19:03 +
Subject: [PATCH] [cfi][CodeGen] Call SetLLVMFunctionAttributes{,ForDefinition}
https://github.com/Atousa updated
https://github.com/llvm/llvm-project/pull/80939
>From ac75fc2873fc7b8eec6c24ba97f4673e94457c8e Mon Sep 17 00:00:00 2001
From: Atousa Duprat
Date: Tue, 6 Feb 2024 21:02:05 -0800
Subject: [PATCH 1/6] [clang] Use separator for large numeric values in
overflow
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
ayermolo wrote:
At least on my mac it passed. With that check.
I double checked, and this is
https://github.com/Atousa updated
https://github.com/llvm/llvm-project/pull/80939
>From ac75fc2873fc7b8eec6c24ba97f4673e94457c8e Mon Sep 17 00:00:00 2001
From: Atousa Duprat
Date: Tue, 6 Feb 2024 21:02:05 -0800
Subject: [PATCH 1/6] [clang] Use separator for large numeric values in
overflow
@@ -942,20 +942,28 @@ CompressedOffloadBundle::compress(const
llvm::MemoryBuffer ,
Input.getBuffer().size());
llvm::compression::Format CompressionFormat;
+ int Level;
- if (llvm::compression::zstd::isAvailable())
+ if (llvm::compression::zstd::isAvailable()) {
HoBoIs wrote:
> DR changes looks good now, save for a nitpick. However, I'm leaving it to
> other reviewers to check contents of the test. It looks like an example from
> the DR, but I don't feel qualified to say whether this test has sufficient
> coverage.
I have added tests to other files
@@ -5241,8 +5241,8 @@ bool CheckVectorElementCallArgs(Sema *S, CallExpr
*TheCall) {
// Note: if we get here one of the args is a scalar which
// requires a VectorSplat on Arg0 or Arg1
- S->Diag(BuiltinLoc, diag::err_vec_builtin_non_vector_all)
- <<
@@ -5218,15 +5218,15 @@ bool CheckVectorElementCallArgs(Sema *S, CallExpr
*TheCall) {
// Note: type promotion is intended to be handeled via the intrinsics
// and not the builtin itself.
S->Diag(TheCall->getBeginLoc(),
-
@@ -22,7 +22,7 @@ float test_dot_vector_size_mismatch(float3 p0, float2 p1) {
float test_dot_builtin_vector_size_mismatch(float3 p0, float2 p1) {
return __builtin_hlsl_dot(p0, p1);
- // expected-error@-1 {{all arguments to '__builtin_hlsl_dot' must have
vectors of the
@@ -5241,8 +5241,8 @@ bool CheckVectorElementCallArgs(Sema *S, CallExpr
*TheCall) {
// Note: if we get here one of the args is a scalar which
// requires a VectorSplat on Arg0 or Arg1
- S->Diag(BuiltinLoc, diag::err_vec_builtin_non_vector_all)
- <<
@@ -22,7 +22,7 @@ float test_dot_vector_size_mismatch(float3 p0, float2 p1) {
float test_dot_builtin_vector_size_mismatch(float3 p0, float2 p1) {
return __builtin_hlsl_dot(p0, p1);
- // expected-error@-1 {{all arguments to '__builtin_hlsl_dot' must have
vectors of the
https://github.com/pow2clk commented:
Just some nitpicks. I also find it much harder to rereview a change when a
force push has gone through. I don't know how things are done around here, but
if it can be avoided, I prefer it.
https://github.com/llvm/llvm-project/pull/83609
https://github.com/pow2clk edited
https://github.com/llvm/llvm-project/pull/83609
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5218,15 +5218,15 @@ bool CheckVectorElementCallArgs(Sema *S, CallExpr
*TheCall) {
// Note: type promotion is intended to be handeled via the intrinsics
// and not the builtin itself.
S->Diag(TheCall->getBeginLoc(),
-
MaskRay wrote:
> Presumably similar things might show up in macros? But can cross that bridge
> when we come to it.
>
> Perhaps we have some/could use some generic utility for this sort of
> contextual warning "if these things are literally written next to each other,
> warn, but if they
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
dwblaikie wrote:
Nah, I think that's probably fine without an explicit `-ggdb` test - though
https://github.com/farzonl updated
https://github.com/llvm/llvm-project/pull/83609
>From bfd8afd06400bc936c13d2138576bb0bb51960e9 Mon Sep 17 00:00:00 2001
From: Farzon Lotfi
Date: Fri, 1 Mar 2024 14:21:17 -0500
Subject: [PATCH] [clang][sema] consolidate diags
---
https://github.com/ayermolo edited
https://github.com/llvm/llvm-project/pull/83331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ayermolo edited
https://github.com/llvm/llvm-project/pull/83331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
ayermolo wrote:
Oh I see!
Thanks for clarification.
That check should be there. I think it
https://github.com/ayermolo updated
https://github.com/llvm/llvm-project/pull/83331
>From 7d58760a887fc935e2af670f8aa721cdd5946391 Mon Sep 17 00:00:00 2001
From: Alexander Yermolovich
Date: Fri, 23 Feb 2024 14:52:04 -0800
Subject: [PATCH 1/2] [CLANG][DWARF] Do not emit -ggnu-pubnames for split
github-actions[bot] wrote:
@rayroudc Congratulations on having your first Pull Request (PR) merged into
the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a
https://github.com/HazardyKnusperkeks closed
https://github.com/llvm/llvm-project/pull/83380
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: rayroudc
Date: 2024-03-01T21:24:13+01:00
New Revision: 76ce3c1bcf82056a61bcbdf776f67ac53d881df0
URL:
https://github.com/llvm/llvm-project/commit/76ce3c1bcf82056a61bcbdf776f67ac53d881df0
DIFF:
https://github.com/llvm/llvm-project/commit/76ce3c1bcf82056a61bcbdf776f67ac53d881df0.diff
@@ -394,33 +412,155 @@ void CGRecordLowering::accumulateFields() {
: getStorageType(*Field),
*Field));
++Field;
-} else {
- ++Field;
}
}
}
-void
-CGRecordLowering::accumulateBitFields(RecordDecl::field_iterator Field,
-
HazardyKnusperkeks wrote:
The question is: why? And why handle c++ different than all other languages?
https://github.com/llvm/llvm-project/pull/83533
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
dwblaikie wrote:
> As discussed in original PR, #82840 , the idea is if -gsplit-dwarf with
>
erichkeane wrote:
We do need a unit test for this, a reduced version of the original fuzzed
result will help us review this.
https://github.com/llvm/llvm-project/pull/77698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
francisvm wrote:
Maybe we can agree to not allow nested `<...>`, and it should simplify the
parsing a little?
https://github.com/llvm/llvm-project/pull/83584
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -38,70 +38,70 @@ float test_dot_element_type_mismatch(int2 p0, float2 p1) {
//NOTE: for all the *_promotion we are intentionally not handling type
promotion in builtins
float test_builtin_dot_vec_int_to_float_promotion(int2 p0, float2 p1) {
return __builtin_hlsl_dot(p0,
https://github.com/ayermolo edited
https://github.com/llvm/llvm-project/pull/83331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dwblaikie wrote:
Presumably similar things might show up in macros? But can cross that bridge
when we come to it.
Perhaps we have some/could use some generic utility for this sort of contextual
warning "if these things are literally written next to each other, warn, but if
they came to be
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
MaskRay wrote:
I haven't checked closely, but you probably need a `-NOT` directive, and if the
@@ -4478,14 +4478,20 @@ renderDebugOptions(const ToolChain , const Driver
, const llvm::Triple ,
Args.getLastArg(options::OPT_ggnu_pubnames,
options::OPT_gno_gnu_pubnames,
options::OPT_gpubnames, options::OPT_gno_pubnames);
if (DwarfFission !=
https://github.com/ayermolo edited
https://github.com/llvm/llvm-project/pull/83331
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
ayermolo wrote:
Sorry not sure I follow.
As discussed in original diff the idea is if
@@ -94,6 +94,36 @@ static llvm::BitVector findReachableBlocks(const CFG ) {
return BlockReachable;
}
+static llvm::DenseSet
+buildContainsExprConsumedInDifferentBlock(
+const CFG ,
+const llvm::DenseMap ) {
+ llvm::DenseSet Result;
+
+ auto CheckChildExprs = [,
@@ -307,7 +310,22 @@ computeBlockInputState(const CFGBlock ,
AnalysisContext ) {
}
}
- JoinedStateBuilder Builder(AC);
+ // If any of the predecessor blocks contains an expression consumed in a
+ // different block, we need to keep expression state.
+ // Note that
@@ -221,18 +221,21 @@ class PrettyStackTraceCFGElement : public
llvm::PrettyStackTraceEntry {
// Avoids unneccesary copies of the environment.
class JoinedStateBuilder {
AnalysisContext
+ Environment::ExprJoinBehavior ExprBehavior;
ymand wrote:
If you're
@@ -248,12 +251,12 @@ class JoinedStateBuilder {
// initialize the state of each basic block differently.
return {AC.Analysis.typeErasedInitialElement(), AC.InitEnv.fork()};
if (All.size() == 1)
ymand wrote:
guard on ExprBehavior as well, like
https://github.com/ymand edited https://github.com/llvm/llvm-project/pull/82611
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ymand approved this pull request.
I'm a little concerned with the complexity and cost required to support this
optimization. With this added cost, do you know if the analysis still faster
(and less SAT timeouts?) than if we just did the simple thing of not dropping
state?
https://github.com/erichkeane updated
https://github.com/llvm/llvm-project/pull/83611
>From 8152ad56b320719553701edf020c30aea8c3213e Mon Sep 17 00:00:00 2001
From: erichkeane
Date: Fri, 1 Mar 2024 11:36:14 -0800
Subject: [PATCH 1/2] Fix null-deref thanks to an attribute on a global
declarator
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 601a9587a1a7eb8dd6377c4ab332edd3bce97a98
8152ad56b320719553701edf020c30aea8c3213e --
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Erich Keane (erichkeane)
Changes
This was reported (sort of) in a PR: #77703. The problem is that a
declarator 'owns' an attributes allocation via an `AttributePool`. However,
this example tries to copy a DeclaratorChunk from one
@@ -38,70 +38,70 @@ float test_dot_element_type_mismatch(int2 p0, float2 p1) {
//NOTE: for all the *_promotion we are intentionally not handling type
promotion in builtins
float test_builtin_dot_vec_int_to_float_promotion(int2 p0, float2 p1) {
return __builtin_hlsl_dot(p0,
https://github.com/erichkeane created
https://github.com/llvm/llvm-project/pull/83611
This was reported (sort of) in a PR: #77703. The problem is that a declarator
'owns' an attributes allocation via an `AttributePool`. However, this example
tries to copy a DeclaratorChunk from one Declarator
philnik777 wrote:
Hm, yeah. I don't think it's worth complicating the parser for a tiny bit of
syntax sugar. I like your idea with a `:` quite a bit. `_ExtVector(bool:4)`
would also be an option. I don't have a strong preference either way. I'd like
to keep it separate from the name though to
fpetrogalli wrote:
Other C++ compatible option: provide a family of templates, one for each number
of lanes, templated on the type:
```
_ExtVector_N
```
So that we need up having:
```
_ExtVector_2
_ExtVector_6
...
```
This would be pretty easy to handle, without introducing extra
https://github.com/python3kgae approved this pull request.
https://github.com/llvm/llvm-project/pull/83609
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -10266,6 +10266,11 @@ def err_block_on_vm : Error<
def err_sizeless_nonlocal : Error<
"non-local variable with sizeless type %0">;
+def err_vec_builtin_non_vector_all : Error<
+ "all arguments to %0 must be vectors">;
+def err_vec_builtin_incompatible_vector_all : Error<
@@ -11,7 +11,6 @@
// NOINLINE-NOT: "-fsplit-dwarf-inlining"
// SPLIT-NOT: "-dumpdir"
// SPLIT: "-debug-info-kind=constructor"
-// SPLIT-SAME: "-ggnu-pubnames"
dwblaikie wrote:
has this lost coverage for non `-glldb` configurations?
(like would we still
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-hlsl
Author: Farzon Lotfi (farzonl)
Changes
removing the additions of `err_vec_builtin_non_vector_all`
`err_vec_builtin_incompatible_vector_all`
caused by https://github.com/llvm/llvm-project/pull/83077.
Instead adding a
https://github.com/farzonl created
https://github.com/llvm/llvm-project/pull/83609
removing the additions of `err_vec_builtin_non_vector_all` &
`err_vec_builtin_incompatible_vector_all`
caused by https://github.com/llvm/llvm-project/pull/83077.
Instead adding a select option to
fpetrogalli wrote:
> > @philnik777 - thank you for the patch at #68324
>
> You're welcome!
>
> FWIW I'd find a syntax like `_ExtVector` better. The underscore and
> upper case to make it clear that it's non-standard and the angle bracket
> syntax since it's kind-of a template. This
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 2b4d67bf59d609321701540a15f48eda04688652
ab12de397fa0c028ad6b92751a31ccabfe89dc7d --
erichkeane wrote:
I spent a while debugging this, and this is far from the correct solution.
However, in my debugging I was able to identify the correct solution, so I'll
prepare a patch to fix this, closing.
https://github.com/llvm/llvm-project/pull/77703
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/77703
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fpetrogalli updated
https://github.com/llvm/llvm-project/pull/83584
>From 9cd3e17192833e2cbafed55b649307ead32dcc3e Mon Sep 17 00:00:00 2001
From: Francesco Petrogalli
Date: Fri, 1 Mar 2024 16:23:57 +0100
Subject: [PATCH 1/4] [clang][Builtins] Parse clang extended vectors
PeterChou1 wrote:
> Hi, you need to edit `clang/include/clang/Format/Format.h` and then run
> `clang/docs/tools/dump_format_style.py` to regenerate
> `clang/docs/ClangFormatStyleOptions.rst`.
Hi my mistake. I'm quite new to the LLVM project so I was not aware the docs
were generated through
@@ -0,0 +1,20 @@
+// RUN: mkdir -p %t/src
+// RUN: cp %s %t/src/debug-info-debug-prefix-map.c
+
+// RUN: mkdir -p %t/out
+// RUN: cd %t/out
+// RUN: %clang_cc1 -debug-info-kind=limited -triple %itanium_abi_triple \
+// RUN: -fdebug-prefix-map="%t/=./"
@@ -394,33 +412,155 @@ void CGRecordLowering::accumulateFields() {
: getStorageType(*Field),
*Field));
++Field;
-} else {
- ++Field;
}
}
}
-void
-CGRecordLowering::accumulateBitFields(RecordDecl::field_iterator Field,
-
jhuber6 wrote:
> It seems being installed twice both under `lib` and
> `lib/x86_64-unknown-linux-gnu`. files are the identical as diff show nothing.
Makes sense, like `add_llvm_library` is implicitly installing it there, then
our subsequent `install` call is doing it again. I wonder if
https://github.com/PeterChou1 updated
https://github.com/llvm/llvm-project/pull/83415
>From 67154ff4388447ff78b2912e5635231778ed23d4 Mon Sep 17 00:00:00 2001
From: PeterChou1 <4355+peterch...@users.noreply.github.com>
Date: Thu, 29 Feb 2024 06:55:18 -0500
Subject: [PATCH 1/2]
ye-luo wrote:
It seems being installed twice both under `lib` and
`lib/x86_64-unknown-linux-gnu`. files are the identical as diff show nothing.
https://github.com/llvm/llvm-project/pull/83282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
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 b542501ad7020bc03437e0c5b12b3e89c05c05af
4fac5b1defe9ce1174da4a2c75f84087f26c63ab --
https://github.com/yxsamliu updated
https://github.com/llvm/llvm-project/pull/83605
>From 6b5687e16c826053d690b08b6fe714e055905479 Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Fri, 1 Mar 2024 13:16:45 -0500
Subject: [PATCH] [HIP] change compress level
Change compression level to 20
jhuber6 wrote:
> ```
> yeluo@epyc-server:/soft/llvm/main-20240301/lib$ ls libomp* -l
> lrwxrwxrwx 1 yeluo yeluo 34 Mar 1 11:18 libomptarget.rtl.amdgpu.so ->
> libomptarget.rtl.amdgpu.so.19.0git
> -r--r--r-- 1 yeluo yeluo 67532024 Mar 1 11:04
> libomptarget.
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
Change compression level to 20 for zstd better
compression rate.
---
Full diff: https://github.com/llvm/llvm-project/pull/83605.diff
4 Files Affected:
- (modified) clang/lib/Driver/OffloadBundler.cpp
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/83605
Change compression level to 20 for zstd better
compression rate.
>From 4fac5b1defe9ce1174da4a2c75f84087f26c63ab Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Fri, 1 Mar 2024 13:16:45 -0500
Subject:
ye-luo wrote:
```
yeluo@epyc-server:/soft/llvm/main-20240301/lib$ ls libomp* -l
lrwxrwxrwx 1 yeluo yeluo 34 Mar 1 11:18 libomptarget.rtl.amdgpu.so ->
libomptarget.rtl.amdgpu.so.19.0git
-r--r--r-- 1 yeluo yeluo 67532024 Mar 1 11:04
libomptarget.rtl.amdgpu.so.19.0git
lrwxrwxrwx 1 ye
https://github.com/labrinea updated
https://github.com/llvm/llvm-project/pull/80540
>From 5354d6c6736f84881466b6b5b99479137eddbf29 Mon Sep 17 00:00:00 2001
From: Alexandros Lamprineas
Date: Tue, 30 Jan 2024 11:17:55 +
Subject: [PATCH] [TargetParser][AArch64] Add alias for FEAT_RDM.
This
labrinea wrote:
Thanks!
https://github.com/llvm/llvm-project/pull/80540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
Message-ID:
In-Reply-To:
https://github.com/Endilll approved this pull request.
DR changes looks good now, save for a nitpick.
However, I'm leaving it to other
Botond =?utf-8?q?Istv=C3=A1n_Horv=C3=A1th?=,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
Message-ID:
In-Reply-To:
@@ -68,3 +68,35 @@ template struct X {};
X<1> x;
#endif
}
+namespace dr2445 { // dr2445: 19
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
Message-ID:
In-Reply-To:
https://github.com/Endilll edited
https://github.com/llvm/llvm-project/pull/83279
___
https://github.com/farzonl closed
https://github.com/llvm/llvm-project/pull/83570
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Farzon Lotfi
Date: 2024-03-01T12:27:25-05:00
New Revision: b542501ad7020bc03437e0c5b12b3e89c05c05af
URL:
https://github.com/llvm/llvm-project/commit/b542501ad7020bc03437e0c5b12b3e89c05c05af
DIFF:
https://github.com/llvm/llvm-project/commit/b542501ad7020bc03437e0c5b12b3e89c05c05af.diff
https://github.com/farzonl edited
https://github.com/llvm/llvm-project/pull/83570
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9442,9 +9442,21 @@ bool
SpecialMemberDeletionInfo::shouldDeleteForSubobjectCall(
int DiagKind = -1;
- if (SMOR.getKind() == Sema::SpecialMemberOverloadResult::NoMemberOrDeleted)
-DiagKind = !Decl ? 0 : 1;
- else if (SMOR.getKind() ==
https://github.com/damyanp approved this pull request.
https://github.com/llvm/llvm-project/pull/83570
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
balazske wrote:
Do you plan to commit this re-apply to the LLVM repository?
https://github.com/llvm/llvm-project/pull/82476
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Bäder
Date: 2024-03-01T18:06:18+01:00
New Revision: 68516bfd2f086736dfd88374a11017276e61ad3d
URL:
https://github.com/llvm/llvm-project/commit/68516bfd2f086736dfd88374a11017276e61ad3d
DIFF:
https://github.com/llvm/llvm-project/commit/68516bfd2f086736dfd88374a11017276e61ad3d.diff
damyanp wrote:
> This chane reuses llvms existing intrinsic `__builtin_elementwise_round`\
> `int_round`
I suspect a typo - `chane` -> `change`?
https://github.com/llvm/llvm-project/pull/83570
___
cfe-commits mailing list
Smertig wrote:
Ping?
How long it usually takes to merge already reviewed PR? Is something else
expected from my side?
https://github.com/llvm/llvm-project/pull/82161
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/MaskRay closed
https://github.com/llvm/llvm-project/pull/83497
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Fangrui Song
Date: 2024-03-01T08:57:54-08:00
New Revision: 64216ba1e427fab1ee38ef9492d3fbca907606b9
URL:
https://github.com/llvm/llvm-project/commit/64216ba1e427fab1ee38ef9492d3fbca907606b9
DIFF:
https://github.com/llvm/llvm-project/commit/64216ba1e427fab1ee38ef9492d3fbca907606b9.diff
@@ -92,6 +92,19 @@ void foo() {
bool is_true = [](){ return true; };
// expected-warning@-1{{address of lambda function pointer conversion
operator will always evaluate to 'true'}}
}
+
+template
+static bool IsFalse(const Ts&...) { return false; }
+template
+static bool
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/83497
>From d7a168190f2fdf3b4f8ec1457400ad8e03bc3f3a Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Thu, 29 Feb 2024 14:40:00 -0800
Subject: [PATCH] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20initia?=
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/83497
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zahiraam updated
https://github.com/llvm/llvm-project/pull/67592
>From 55b67a58ef8b9856e5f0a8f535b8617f59711dec Mon Sep 17 00:00:00 2001
From: Ammarguellat
Date: Wed, 27 Sep 2023 11:59:04 -0700
Subject: [PATCH 01/20] Fix value of __FUNCTION__ and __func__ in MSVC mode.
---
@@ -831,6 +831,40 @@ uint64_t3 reversebits(uint64_t3);
_HLSL_BUILTIN_ALIAS(__builtin_elementwise_bitreverse)
uint64_t4 reversebits(uint64_t4);
+//===--===//
+// frac builtins
https://github.com/farzonl updated
https://github.com/llvm/llvm-project/pull/83570
>From ca14584d500752215b922e18743a503ea7f323bc Mon Sep 17 00:00:00 2001
From: Farzon Lotfi
Date: Thu, 29 Feb 2024 21:48:47 -0500
Subject: [PATCH] [HLSL][DXIL] Implementation of round intrinsic
hlsl_intrinsics.h
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 5b6e58c565cf809e4133a10ff9d9b096754bea1e
f9e5cf8043dffc0c9fc94a8768b25111664311dd --
https://github.com/farzonl updated
https://github.com/llvm/llvm-project/pull/83570
>From f9e5cf8043dffc0c9fc94a8768b25111664311dd Mon Sep 17 00:00:00 2001
From: Farzon Lotfi
Date: Thu, 29 Feb 2024 21:48:47 -0500
Subject: [PATCH] [HLSL][DXIL] Implementation of round intrinsic
hlsl_intrinsics.h
https://github.com/jroelofs approved this pull request.
https://github.com/llvm/llvm-project/pull/80540
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,43 @@
+; RUN: opt -S -dxil-op-lower < %s | FileCheck %s
+
+; Make sure dxil operation function calls for round are generated for float
and half.
+; CHECK:call float @dx.op.unary.f32(i32 26, float %{{.*}})
+; CHECK:call half @dx.op.unary.f16(i32 26, half %{{.*}})
+
@@ -0,0 +1,43 @@
+; RUN: opt -S -dxil-op-lower < %s | FileCheck %s
+
+; Make sure dxil operation function calls for round are generated for float
and half.
+; CHECK:call float @dx.op.unary.f32(i32 26, float %{{.*}})
+; CHECK:call half @dx.op.unary.f16(i32 26, half %{{.*}})
+
@@ -0,0 +1,43 @@
+; RUN: opt -S -dxil-op-lower < %s | FileCheck %s
+
+; Make sure dxil operation function calls for round are generated for float
and half.
+; CHECK:call float @dx.op.unary.f32(i32 26, float %{{.*}})
+; CHECK:call half @dx.op.unary.f16(i32 26, half %{{.*}})
+
@@ -220,6 +220,9 @@ def Sin : DXILOpMapping<13, unary, int_sin,
"Returns sine(theta) for theta in radians.">;
def UMax : DXILOpMapping<39, binary, int_umax,
"Unsigned integer maximum. UMax(a,b) = a > b ? a :
b">;
+def Round :
101 - 200 of 365 matches
Mail list logo