Skip to content

Conversation

@bogner
Copy link
Contributor

@bogner bogner commented Apr 4, 2025

No description provided.

@bogner bogner merged commit 77cfa38 into llvm:main Apr 4, 2025
6 of 10 checks passed
@llvmbot
Copy link
Member

llvmbot commented Apr 4, 2025

@llvm/pr-subscribers-backend-directx

Author: Justin Bogner (bogner)

Changes

Full diff: https://github.com/llvm/llvm-project/pull/134379.diff

1 Files Affected:

  • (modified) llvm/lib/Target/DirectX/DirectXTargetTransformInfo.cpp (+7-7)
diff --git a/llvm/lib/Target/DirectX/DirectXTargetTransformInfo.cpp b/llvm/lib/Target/DirectX/DirectXTargetTransformInfo.cpp
index ba656dc737140..765e1977041b9 100644
--- a/llvm/lib/Target/DirectX/DirectXTargetTransformInfo.cpp
+++ b/llvm/lib/Target/DirectX/DirectXTargetTransformInfo.cpp
@@ -38,18 +38,18 @@ bool DirectXTTIImpl::isTargetIntrinsicWithOverloadTypeAtArg(Intrinsic::ID ID,
 bool DirectXTTIImpl::isTargetIntrinsicTriviallyScalarizable(
     Intrinsic::ID ID) const {
   switch (ID) {
+  case Intrinsic::dx_asdouble:
+  case Intrinsic::dx_firstbitlow:
+  case Intrinsic::dx_firstbitshigh:
+  case Intrinsic::dx_firstbituhigh:
   case Intrinsic::dx_frac:
   case Intrinsic::dx_rsqrt:
+  case Intrinsic::dx_splitdouble:
+  case Intrinsic::dx_wave_readlane:
   case Intrinsic::dx_wave_reduce_max:
-  case Intrinsic::dx_wave_reduce_umax:
   case Intrinsic::dx_wave_reduce_sum:
+  case Intrinsic::dx_wave_reduce_umax:
   case Intrinsic::dx_wave_reduce_usum:
-  case Intrinsic::dx_wave_readlane:
-  case Intrinsic::dx_asdouble:
-  case Intrinsic::dx_splitdouble:
-  case Intrinsic::dx_firstbituhigh:
-  case Intrinsic::dx_firstbitshigh:
-  case Intrinsic::dx_firstbitlow:
     return true;
   default:
     return false;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants