@@ -831,6 +831,40 @@ uint64_t3 reversebits(uint64_t3);
_HLSL_BUILTIN_ALIAS(__builtin_elementwise_bitreverse)
uint64_t4 reversebits(uint64_t4);
+//===--===//
+// frac builtins
@@ -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 %{{.*}})
+
https://github.com/bharadwajy approved this pull request.
A few simple suggestions. Otherwise looks good.
https://github.com/llvm/llvm-project/pull/83570
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/bharadwajy 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
https://github.com/mzyKi updated https://github.com/llvm/llvm-project/pull/83585
>From 6a9f7a996c127462fa787a2626bc0676d4e04ff3 Mon Sep 17 00:00:00 2001
From: miaozhiyuan
Date: Fri, 1 Mar 2024 22:45:20 +0800
Subject: [PATCH] [clang][ExprEngineCXX] Fix crash on dereference invalid
return value
https://github.com/mzyKi edited https://github.com/llvm/llvm-project/pull/83585
___
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
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Exile (mzyKi)
Changes
fix #78810
---
Full diff: https://github.com/llvm/llvm-project/pull/83585.diff
2 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp (+7-2)
-
philnik777 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 unfortunately doesn't
https://github.com/mzyKi created https://github.com/llvm/llvm-project/pull/83585
fix #78810
>From f0291dde10251269627022a9b7331211a3d6a91f Mon Sep 17 00:00:00 2001
From: miaozhiyuan
Date: Fri, 1 Mar 2024 22:45:20 +0800
Subject: [PATCH] [clang][ExprEngineCXX] Fix crash on dereference invalid
Author: Joseph Huber
Date: 2024-03-01T09:48:14-06:00
New Revision: b92c3fe0274f3ba3bb7c58a8529bd9c4303a3b51
URL:
https://github.com/llvm/llvm-project/commit/b92c3fe0274f3ba3bb7c58a8529bd9c4303a3b51
DIFF:
https://github.com/llvm/llvm-project/commit/b92c3fe0274f3ba3bb7c58a8529bd9c4303a3b51.diff
Author: Vinayak Dev
Date: 2024-03-01T10:40:46-05:00
New Revision: 2b4d67bf59d609321701540a15f48eda04688652
URL:
https://github.com/llvm/llvm-project/commit/2b4d67bf59d609321701540a15f48eda04688652
DIFF:
https://github.com/llvm/llvm-project/commit/2b4d67bf59d609321701540a15f48eda04688652.diff
AaronBallman wrote:
> Hi @cor3ntin , I tried but didn't work as per design, Can you pls review this
> , I m not updating the Decl , updating the VarDecl
I'd like more information on what was tried and why it didn't work. It would be
better for us to associate the data with catch statements
https://github.com/jhuber6 closed
https://github.com/llvm/llvm-project/pull/83573
___
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 2b4d67bf59d609321701540a15f48eda04688652 Mon Sep 17 00:00:00 2001
From: Vinayak Dev <104419489+vinayakd...@users.noreply.github.com>
Date: Fri, 1 Mar 2024 21:10:46 +0530
Subject: [PATCH 1/3]
https://github.com/bjope approved this pull request.
https://github.com/llvm/llvm-project/pull/83573
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Erich Keane
Date: 2024-03-01T07:27:06-08:00
New Revision: 06bd74ba4ac5229f01b64772b49e025be5eb7b53
URL:
https://github.com/llvm/llvm-project/commit/06bd74ba4ac5229f01b64772b49e025be5eb7b53
DIFF:
https://github.com/llvm/llvm-project/commit/06bd74ba4ac5229f01b64772b49e025be5eb7b53.diff
@@ -101,17 +101,6 @@
/// ###
-/// Check that the warning is thrown when the libomptarget bitcode library is
not found.
-/// Libomptarget requires sm_52 or newer so an sm_52 bitcode library should
never
Author: Timm Bäder
Date: 2024-03-01T16:19:45+01:00
New Revision: a038f9758e02812803b7efce10ecf784f9842bbb
URL:
https://github.com/llvm/llvm-project/commit/a038f9758e02812803b7efce10ecf784f9842bbb
DIFF:
https://github.com/llvm/llvm-project/commit/a038f9758e02812803b7efce10ecf784f9842bbb.diff
Author: Erich Keane
Date: 2024-03-01T07:11:17-08:00
New Revision: b0181be36cace3460e4ec5d0d11ecbf49484cc55
URL:
https://github.com/llvm/llvm-project/commit/b0181be36cace3460e4ec5d0d11ecbf49484cc55
DIFF:
https://github.com/llvm/llvm-project/commit/b0181be36cace3460e4ec5d0d11ecbf49484cc55.diff
@@ -101,17 +101,6 @@
/// ###
-/// Check that the warning is thrown when the libomptarget bitcode library is
not found.
-/// Libomptarget requires sm_52 or newer so an sm_52 bitcode library should
never
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/81127
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fpetrogalli edited
https://github.com/llvm/llvm-project/pull/83584
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -14478,7 +14478,7 @@ C++ defect report implementation
status
https://cplusplus.github.io/CWG/issues/2445.html;>2445
C++20
Partial ordering with rewritten candidates
-Unknown
+Clang 18
HoBoIs wrote:
done
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/tbaederr updated
https://github.com/llvm/llvm-project/pull/83279
>From 68200ecf3267d1b3940fa73c25c50ee706932a98 Mon Sep
fpetrogalli wrote:
@philnik777 - thank you for the patch at
https://github.com/llvm/llvm-project/pull/68324
I am extending the parser to be able to recognise clang extended vectors.
Thanks!
Francesco
https://github.com/llvm/llvm-project/pull/83584
Author: Timm Bäder
Date: 2024-03-01T15:46:00+01:00
New Revision: f15d799f16092918b948536775475dfd8675c7d9
URL:
https://github.com/llvm/llvm-project/commit/f15d799f16092918b948536775475dfd8675c7d9
DIFF:
https://github.com/llvm/llvm-project/commit/f15d799f16092918b948536775475dfd8675c7d9.diff
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Francesco Petrogalli (fpetrogalli)
Changes
Clang extended vector types are mangled as follows:
ext_vector_type_lanes_scalar type
This is used to defetmine the builtins signature for builtins that use
parmeters defined as
typedef
Munesanz wrote:
> > This patch is now only for the front end. @shiltian @alexey-bataev, let us
> > know what you think about landing just this. We are going to push another
> > PR with the runtime changes, and that one will include numbers for
> > performance.
>
> Hi! Can someone post a link
https://github.com/fpetrogalli created
https://github.com/llvm/llvm-project/pull/83584
Clang extended vector types are mangled as follows:
ext_vector_type__
This is used to defetmine the builtins signature for builtins that use
parmeters defined as
typedef ext_vector_type__
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
Message-ID:
In-Reply-To:
@@ -14478,7 +14478,7 @@ C++ defect report implementation
status
https://cplusplus.github.io/CWG/issues/2445.html;>2445
C++20
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/83487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/83487
>From 1c443bec9f11c14f8971d5dcb03d12e50c080afc Mon Sep 17 00:00:00 2001
From: erichkeane
Date: Thu, 29 Feb 2024 13:29:02 -0800
Subject: [PATCH 1/3] Fix implementation of [temp.param]p14's first sentence.
The
Author: Timm Bäder
Date: 2024-03-01T15:41:49+01:00
New Revision: f651f134bbaec069968f6b12bdcdb5f7752fd700
URL:
https://github.com/llvm/llvm-project/commit/f651f134bbaec069968f6b12bdcdb5f7752fd700
DIFF:
https://github.com/llvm/llvm-project/commit/f651f134bbaec069968f6b12bdcdb5f7752fd700.diff
@@ -101,17 +101,6 @@
/// ###
-/// Check that the warning is thrown when the libomptarget bitcode library is
not found.
-/// Libomptarget requires sm_52 or newer so an sm_52 bitcode library should
never
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/83487
>From 1c443bec9f11c14f8971d5dcb03d12e50c080afc Mon Sep 17 00:00:00 2001
From: erichkeane
Date: Thu, 29 Feb 2024 13:29:02 -0800
Subject: [PATCH 1/3] Fix implementation of [temp.param]p14's first sentence.
The
Author: Timm Bäder
Date: 2024-03-01T15:40:45+01:00
New Revision: e59681d96327e2ed1963ec1c0f2bc3d40df26443
URL:
https://github.com/llvm/llvm-project/commit/e59681d96327e2ed1963ec1c0f2bc3d40df26443
DIFF:
https://github.com/llvm/llvm-project/commit/e59681d96327e2ed1963ec1c0f2bc3d40df26443.diff
vinayakdsci wrote:
Great catch! The usage of template instantiation didn't seem very obvious while
adding the tests, and hence the missed test case.
Thanks a lot for the fix!
https://github.com/llvm/llvm-project/pull/83497
___
cfe-commits mailing
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/83460
___
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
Message-ID:
In-Reply-To:
@@ -5548,13 +5504,100 @@ static bool isAtLeastAsSpecializedAs(Sema ,
FunctionTemplateDecl *Sema::getMoreSpecializedTemplate(
https://github.com/DavidSpickett edited
https://github.com/llvm/llvm-project/pull/81869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
Thanks for the details, then LGTM with a release note added when landing.
https://github.com/llvm/llvm-project/pull/83487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Endilll wrote:
My plan is to add tests to this PR, but if you want to land this sooner, I can
do tests in a separate PR.
https://github.com/llvm/llvm-project/pull/83487
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
erichkeane wrote:
> The changes look correct to me, but I think we should add a DR test for this
> so that we can properly regenerate cxx_dr_status.html
>
> Also, the changes should come with a release note.
Ooof, yeah, forgot the release note. @Endilll has volunteered to do the DR
test in
https://github.com/HoBoIs edited https://github.com/llvm/llvm-project/pull/83279
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -14478,7 +14478,7 @@ C++ defect report implementation
status
https://cplusplus.github.io/CWG/issues/2445.html;>2445
C++20
Partial ordering with rewritten candidates
-Unknown
+Clang 18
HoBoIs wrote:
I'm on it. While we are at here:
TerryLWilmarth wrote:
> This patch is now only for the front end. @shiltian @alexey-bataev, let us
> know what you think about landing just this. We are going to push another PR
> with the runtime changes, and that one will include numbers for performance.
Hi! Can someone post a link here to
https://github.com/python3kgae 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
AaronBallman wrote:
One question I have is related to:
> Last year MSVC added [[msvc::intrinsic]] for us game devs
> [here](https://github.com/MicrosoftDocs/cpp-docs/blob/main/docs/cpp/attributes.md#msvcintrinsic).
> This was explicitly added as an attribute under the request of us since a
>
yxsamliu wrote:
in HIP headers, warpSize is defined with __AMDGCN_WAVEFRONT_SIZE and there are
a bunch of uses of __AMDGCN_WAVEFRONT_SIZE or warpSize as constants:
https://github.com/search?q=repo%3AROCm%2Fclr%20__AMDGCN_WAVEFRONT_SIZE=code
These can be fixed relatively easily by conditioning
@@ -831,6 +831,40 @@ uint64_t3 reversebits(uint64_t3);
_HLSL_BUILTIN_ALIAS(__builtin_elementwise_bitreverse)
uint64_t4 reversebits(uint64_t4);
+//===--===//
+// frac builtins
python3kgae
jhuber6 wrote:
> Problem 1 can be solved by flipping the order. But Problem 2 would remain as
> it doesn't depend on the order.
https://github.com/llvm/llvm-project/pull/83573 I made a patch to fix it.
https://github.com/llvm/llvm-project/pull/83282
@@ -1847,19 +1847,33 @@ floating point semantic models: precise (the default),
strict, and fast.
* ``16`` - Forces ``_Float16`` operations to be emitted without using excess
precision arithmetic.
-.. option:: -fcx-limited-range:
-
- This option enables the naive
Author: Timm Bäder
Date: 2024-03-01T15:28:15+01:00
New Revision: dbf3d779bdb3cc22652b6ab24ac9827e9f228f4e
URL:
https://github.com/llvm/llvm-project/commit/dbf3d779bdb3cc22652b6ab24ac9827e9f228f4e
DIFF:
https://github.com/llvm/llvm-project/commit/dbf3d779bdb3cc22652b6ab24ac9827e9f228f4e.diff
Author: Timm Bäder
Date: 2024-03-01T15:28:14+01:00
New Revision: 18d2ff4be7898eaf666564dcca07ad6bd38ababf
URL:
https://github.com/llvm/llvm-project/commit/18d2ff4be7898eaf666564dcca07ad6bd38ababf
DIFF:
https://github.com/llvm/llvm-project/commit/18d2ff4be7898eaf666564dcca07ad6bd38ababf.diff
yxsamliu wrote:
for example, rocprim assumes warpSize is constant
https://github.com/ROCm/rocPRIM/blob/6325547d514b46d1ab51aff0195851b3fcc626d1/rocprim/include/rocprim/intrinsics/thread.hpp#L54
since device_warp_size() is used as non-type template arguments and these code
are not conditioned
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/83065
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nikolas Klauser
Date: 2024-03-01T15:15:11+01:00
New Revision: b873847a53ae638e2146e3657fe33efe30c2afe1
URL:
https://github.com/llvm/llvm-project/commit/b873847a53ae638e2146e3657fe33efe30c2afe1
DIFF:
philnik777 wrote:
The CI failure is unrelated.
https://github.com/llvm/llvm-project/pull/83065
___
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: Joseph Huber (jhuber6)
Changes
Summary:
We still use this bitcode library in one case, the NVPTX non-LTO build.
The patch updated the search paths to treat it the same as other
libraries, which unintentionally prioritized system paths over
https://github.com/jhuber6 created
https://github.com/llvm/llvm-project/pull/83573
Summary:
We still use this bitcode library in one case, the NVPTX non-LTO build.
The patch updated the search paths to treat it the same as other
libraries, which unintentionally prioritized system paths over
=?utf-8?q?Donát?= Nagy ,NagyDonat
Message-ID:
In-Reply-To:
@@ -27,20 +27,48 @@ class IdentifierInfo;
namespace clang {
namespace ento {
-
-enum CallDescriptionFlags : unsigned {
- CDF_None = 0,
-
- /// Describes a C standard function that is sometimes implemented as a
@@ -1369,13 +1369,50 @@ class TargetInfo : public TransferrableTargetInfo,
}
struct BranchProtectionInfo {
-LangOptions::SignReturnAddressScopeKind SignReturnAddr =
-LangOptions::SignReturnAddressScopeKind::None;
-LangOptions::SignReturnAddressKeyKind
@@ -27,20 +27,48 @@ class IdentifierInfo;
namespace clang {
namespace ento {
-
-enum CallDescriptionFlags : unsigned {
- CDF_None = 0,
-
- /// Describes a C standard function that is sometimes implemented as a macro
- /// that expands to a compiler builtin with some
@@ -4306,10 +4306,10 @@
// Begin amdgcn tests
-// RUN: %clang -mcpu=gfx803 -E -dM %s -o - 2>&1 \
+// RUN: %clang -mcpu=gfx803 -E -dM -Xclang -fcuda-is-device %s -o - 2>&1 \
yxsamliu wrote:
C code compiled with target amdgcn should not
jhuber6 wrote:
> Problem 1 can be solved by flipping the order. But Problem 2 would remain as
> it doesn't depend on the order.
Honestly, we should just remove the second test. We just treat these things as
libraries and it doesn't make sense for a test to ensure that `-lstdc++`
doesn't
@@ -27,20 +27,48 @@ class IdentifierInfo;
namespace clang {
namespace ento {
-
-enum CallDescriptionFlags : unsigned {
- CDF_None = 0,
-
- /// Describes a C standard function that is sometimes implemented as a macro
- /// that expands to a compiler builtin with some
@@ -6,32 +6,32 @@
// R600-based processors.
//
-// RUN: %clang -E -dM -target r600 -mcpu=r600 %s 2>&1 | FileCheck
--check-prefixes=ARCH-R600,R600 %s -DCPU=r600
-// RUN: %clang -E -dM -target r600 -mcpu=rv630 %s 2>&1 | FileCheck
--check-prefixes=ARCH-R600,R600 %s -DCPU=r600
yxsamliu wrote:
Did you try this patch with internal PSDB? This will likely break all HIP
programs.
This is because HIP is single source program and users usually expect the
common device-side predefined macros is seen in both host and device
compilations. e.g. they could write a kernel
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
Message-ID:
In-Reply-To:
@@ -14478,7 +14478,7 @@ C++ defect report implementation
status
https://cplusplus.github.io/CWG/issues/2445.html;>2445
C++20
Botond =?utf-8?q?Istv=C3=A1n_Horv=C3=A1th?=,Botond Istvan Horvath
,Botond Istvan Horvath
,Botond Istvan Horvath
Message-ID:
In-Reply-To:
https://github.com/Endilll requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/83279
Botond =?utf-8?q?István_Horváth?=,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
___
cfe-commits mailing list
Pierre-vh wrote:
> This was the original behavior of my patch, but I reverted it because it
> broke all the HIP headers that were unintentionally relying on this. Has that
> been resolved?
Was an issue opened for that? How many headers are affected?
jhuber6 wrote:
This was the original behavior of my patch, but I reverted it because it broke
all the HIP headers that were unintentionally relying on this. Has that been
resolved?
https://github.com/llvm/llvm-project/pull/83558
___
cfe-commits
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
llvmbot wrote:
@llvm/pr-subscribers-hlsl
@llvm/pr-subscribers-backend-directx
Author: Farzon Lotfi (farzonl)
Changes
hlsl_intrinsics.h - add the round api
DXIL.td add the llvm intrinsic to DXIL lowering mapping
This chane reuses llvms existing `__builtin_elementwise_round`\`int_round`
https://github.com/farzonl created
https://github.com/llvm/llvm-project/pull/83570
hlsl_intrinsics.h - add the round api
DXIL.td add the llvm intrinsic to DXIL lowering mapping
This chane reuses llvms existing `__builtin_elementwise_round`\`int_round`
This change implements: #70077
>From
bjope wrote:
Problem 1 can be solved by flipping the order.
But Problem 2 would remain as it doesn't depend on the order.
https://github.com/llvm/llvm-project/pull/83282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
Author: None (SuHo-llrr)
Changes
Emit Zicfiss/Zicfilp to .note.gnu.property sections
1. for spec v0.4.0 Zicifss/Zicfilp is AND feature means that all objects need
to have this feature.
2. Emit note section when Zicifss/Zicfilp extension
https://github.com/HoBoIs updated
https://github.com/llvm/llvm-project/pull/83279
From 68200ecf3267d1b3940fa73c25c50ee706932a98 Mon Sep 17 00:00:00 2001
From: Botond Istvan Horvath
Date: Wed, 28 Feb 2024 13:09:15 +0100
Subject: [PATCH 1/5] Bugfix for choosing the more specialized overload
https://github.com/kito-cheng edited
https://github.com/llvm/llvm-project/pull/77414
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
> Hi @jhuber6, @MaskRay
>
> We are having some problems with this patch on a server where the file
> /lib64/libomptarget-nvptx-sm_52.bc exists. The test case that fails is
> clang/test/Driver/openmp-offload-gpu.c.
>
> **Problem 1** I think one problem is related to this check
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
@@ -301,21 +301,27 @@ compareValueToThreshold(ProgramStateRef State, NonLoc
Value, NonLoc Threshold,
// calling `evalBinOpNN`:
if (isNegative(SVB, State, Value) && isUnsigned(SVB, Threshold)) {
if (CheckEquality) {
-
https://github.com/vapdrs updated
https://github.com/llvm/llvm-project/pull/83476
>From f66254c6be64a7270ce2df8556c540344ade0619 Mon Sep 17 00:00:00 2001
From: Douglas Deslauriers
Date: Thu, 29 Feb 2024 20:18:34 +
Subject: [PATCH] [clang] Sequence C++20 Parenthesized List Init
@@ -202,12 +207,20 @@ RValue
CodeGenFunction::EmitAMDGPUDevicePrintfCallExpr(const CallExpr *E) {
Args.push_back(Arg);
}
- llvm::IRBuilder<> IRB(Builder.GetInsertBlock(), Builder.GetInsertPoint());
- IRB.SetCurrentDebugLocation(Builder.getCurrentDebugLocation());
+
https://github.com/AaronBallman commented:
The changes look correct to me, but I think we should add a DR test for this so
that we can properly regenerate cxx_dr_status.html
Also, the changes should come with a release note.
https://github.com/llvm/llvm-project/pull/83487
@@ -2550,6 +2550,11 @@ RValue CodeGenFunction::EmitBuiltinExpr(const GlobalDecl
GD, unsigned BuiltinID,
().getLongDoubleFormat() == ::APFloat::IEEEquad())
BuiltinID = mutateLongDoubleBuiltin(BuiltinID);
+ // Mutate the printf builtin ID so that we use the same
@@ -3616,6 +3617,12 @@ unsigned FunctionDecl::getBuiltinID(bool
ConsiderWrapperFunctions) const {
if (!ConsiderWrapperFunctions && getStorageClass() == SC_Static)
return 0;
+ // AMDGCN implementation supports printf as a builtin
+ // for OpenCL
+ if
https://github.com/AaronBallman approved this pull request.
LGTM with a minor commenting nit. Thank you for catching this and the quick
patch!
https://github.com/llvm/llvm-project/pull/83497
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -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/AaronBallman 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/vapdrs edited https://github.com/llvm/llvm-project/pull/83476
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Pierre-vh updated
https://github.com/llvm/llvm-project/pull/83558
>From 3730631ac58425f559f4bc3cfe3da89e6367c1c5 Mon Sep 17 00:00:00 2001
From: pvanhout
Date: Fri, 1 Mar 2024 12:43:55 +0100
Subject: [PATCH 1/2] [clang][AMDGPU] Don't define feature macros on host code
Those
labrinea wrote:
ping
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
https://github.com/labrinea updated
https://github.com/llvm/llvm-project/pull/80540
>From d2c973c8ebd7605b47a8c5fc928d2d85426c8a6d 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
https://github.com/kito-cheng commented:
Could you add a testcase?
https://github.com/llvm/llvm-project/pull/83553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -301,21 +301,27 @@ compareValueToThreshold(ProgramStateRef State, NonLoc
Value, NonLoc Threshold,
// calling `evalBinOpNN`:
if (isNegative(SVB, State, Value) && isUnsigned(SVB, Threshold)) {
if (CheckEquality) {
- // negative_value == unsigned_value is always
rayroudc wrote:
@owenca, thanks for the review!
As I do not have write access, would it be possible for you to merge this
change ?
https://github.com/llvm/llvm-project/pull/83380
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -292,8 +292,14 @@ void AMDGPUTargetInfo::getTargetDefines(const LangOptions
,
}
Builder.defineMacro(Twine("__") + Twine(CanonName) + Twine("__"));
+
+ // Don't emit feature macros in host code because in such cases the
+ // feature list is not accurate.
+ if
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Pierre van Houtryve (Pierre-vh)
Changes
Those macros are unreliable because our features are mostly uninitialized at
that stage, so any macro we define is unreliable.
Fixes SWDEV-447308
---
Full diff:
https://github.com/Pierre-vh created
https://github.com/llvm/llvm-project/pull/83558
Those macros are unreliable because our features are mostly uninitialized at
that stage, so any macro we define is unreliable.
Fixes SWDEV-447308
>From 3730631ac58425f559f4bc3cfe3da89e6367c1c5 Mon Sep 17
201 - 300 of 365 matches
Mail list logo