[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

https://github.com/aabysswalker updated https://github.com/llvm/llvm-project/pull/86932 >From caec748f3f0ab16b829f7765c5c2d87ed7f7b73f Mon Sep 17 00:00:00 2001 From: aabysswalker Date: Thu, 28 Mar 2024 12:50:25 +0200 Subject: [PATCH 1/2] Prevent generating double of ceil, exp, exp2, floor.

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

@@ -1,12 +1,13 @@ -// RUN: %clang_cc1 -finclude-default-header -triple dxil-pc-shadermodel6.6-library %s -fnative-half-type -emit-llvm -disable-llvm-passes -verify -DTEST_FUNC=__builtin_elementwise_cos -// RUN: %clang_cc1 -finclude-default-header -triple

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

@@ -1,12 +1,13 @@ -// RUN: %clang_cc1 -finclude-default-header -triple dxil-pc-shadermodel6.6-library %s -fnative-half-type -emit-llvm -disable-llvm-passes -verify -DTEST_FUNC=__builtin_elementwise_cos -// RUN: %clang_cc1 -finclude-default-header -triple

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

@@ -1,12 +1,13 @@ -// RUN: %clang_cc1 -finclude-default-header -triple dxil-pc-shadermodel6.6-library %s -fnative-half-type -emit-llvm -disable-llvm-passes -verify -DTEST_FUNC=__builtin_elementwise_cos -// RUN: %clang_cc1 -finclude-default-header -triple

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

farzonl wrote: So keep in mind there are some double cases that are valid for hlsl. so always check via dxc which can be easily used via [hlsl.godbolt](https://hlsl.godbolt.org/). https://github.com/llvm/llvm-project/pull/86932 ___ cfe-commits

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

@@ -1,12 +1,13 @@ -// RUN: %clang_cc1 -finclude-default-header -triple dxil-pc-shadermodel6.6-library %s -fnative-half-type -emit-llvm -disable-llvm-passes -verify -DTEST_FUNC=__builtin_elementwise_cos -// RUN: %clang_cc1 -finclude-default-header -triple

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

farzonl wrote: I was writing up a ticket for floor and ceil. Nice catch! https://godbolt.org/z/q9a644e6W. https://github.com/llvm/llvm-project/pull/86932 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

llvmbot wrote: @llvm/pr-subscribers-hlsl @llvm/pr-subscribers-clang Author: Andrii Levitskiy (aabysswalker) Changes As #86555, we should cover all of non-double builtins. Closes #86818 --- Full diff: https://github.com/llvm/llvm-project/pull/86932.diff 5 Files Affected: - (modified)

[clang] [HLSL] prevent generation of double intrinsics. (PR #86932)

https://github.com/aabysswalker created https://github.com/llvm/llvm-project/pull/86932 As #86555, we should cover all of non-double builtins. Closes #86818 >From caec748f3f0ab16b829f7765c5c2d87ed7f7b73f Mon Sep 17 00:00:00 2001 From: aabysswalker Date: Thu, 28 Mar 2024 12:50:25 +0200