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.
Rem
@@ -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
dxil-pc-shadermodel6.6
@@ -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
dxil-pc-shadermodel6.6
@@ -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
dxil-pc-shadermodel6.6
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 maili
@@ -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
dxil-pc-shadermodel6.6
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
https://lists.llvm.org/cgi-bin/mailman/l
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)
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
Subjec
