Skip to content

Conversation

@bogner
Copy link
Contributor

@bogner bogner commented Aug 3, 2024

As of cdfd884 "[DXIL] Add DXIL version-specific TableGen specification and implementation of DXIL Ops (#97593)", all of these tests need to specify triples.

As of cdfd884 "[DXIL] Add DXIL version-specific TableGen specification and
implementation of DXIL Ops (llvm#97593)", all of these tests need to specify
triples.
@llvmbot
Copy link
Member

llvmbot commented Aug 3, 2024

@llvm/pr-subscribers-backend-directx

Author: Justin Bogner (bogner)

Changes

As of cdfd884 "[DXIL] Add DXIL version-specific TableGen specification and implementation of DXIL Ops (#97593)", all of these tests need to specify triples.


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

2 Files Affected:

  • (modified) llvm/test/CodeGen/DirectX/any.ll (+1-1)
  • (modified) llvm/test/CodeGen/DirectX/lerp.ll (+1-1)
diff --git a/llvm/test/CodeGen/DirectX/any.ll b/llvm/test/CodeGen/DirectX/any.ll
index e8d87075d65cf..ceb8077af311d 100644
--- a/llvm/test/CodeGen/DirectX/any.ll
+++ b/llvm/test/CodeGen/DirectX/any.ll
@@ -1,4 +1,4 @@
-; RUN: opt -S -dxil-op-lower < %s | FileCheck %s
+; RUN: opt -S -dxil-op-lower -mtriple=dxil-pc-shadermodel6.0-library < %s | FileCheck %s
 
 ; Make sure dxil operation function calls for any are generated for float and half.
 
diff --git a/llvm/test/CodeGen/DirectX/lerp.ll b/llvm/test/CodeGen/DirectX/lerp.ll
index ebd7e133b5163..9f5e4c7710d21 100644
--- a/llvm/test/CodeGen/DirectX/lerp.ll
+++ b/llvm/test/CodeGen/DirectX/lerp.ll
@@ -1,4 +1,4 @@
-; RUN: opt -S -dxil-op-lower < %s | FileCheck %s
+; RUN: opt -S -dxil-op-lower -mtriple=dxil-pc-shadermodel6.0-library < %s | FileCheck %s
 
 ; Make sure dxil operation function calls for lerp are generated for float and half.
 

@bogner bogner merged commit 1c7540c into llvm:main Aug 3, 2024
@bogner bogner deleted the 2024-08-02-triples branch August 3, 2024 00:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants