Skip to content

[NVPTX][Test-only] Add proper sm-version to ptxas-verify in prefetch-inferas-test.ll #152492

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

abhilash1910
Copy link
Contributor

@abhilash1910 abhilash1910 commented Aug 7, 2025

prefetch-inferas-test.ll was added in #146203 , but due to missing ptxas version the CI is defaulting to sm 60.
This patch adds the arg in ptxas check.

cc @durga4github

@llvmbot
Copy link
Member

llvmbot commented Aug 7, 2025

@llvm/pr-subscribers-backend-nvptx

Author: Abhilash Majumder (abhilash1910)

Changes

prefetch-inferas-test.ll was added in #146203 , but due to missing ptxas version the CI is defaulting to sm 60.
This patch adds the arg in ptxas check.

cc @durga4github


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/NVPTX/prefetch-inferas-test.ll (+2-1)
diff --git a/llvm/test/CodeGen/NVPTX/prefetch-inferas-test.ll b/llvm/test/CodeGen/NVPTX/prefetch-inferas-test.ll
index 822a75eab24c2..af581e63e4e49 100644
--- a/llvm/test/CodeGen/NVPTX/prefetch-inferas-test.ll
+++ b/llvm/test/CodeGen/NVPTX/prefetch-inferas-test.ll
@@ -1,6 +1,6 @@
 ; RUN: opt < %s -S -passes=infer-address-spaces | FileCheck %s --check-prefix=INFER
 ; RUN: llc < %s -mtriple=nvptx64 -mcpu=sm_90 -mattr=+ptx80 | FileCheck %s --check-prefix=PTX
-; RUN: %if ptxas %{ llc < %s -mtriple=nvptx64 -mcpu=sm_90 -mattr=+ptx80 | %ptxas-verify %}
+; RUN: %if ptxas %{ llc < %s -mtriple=nvptx64 -mcpu=sm_90 -mattr=+ptx80 | %ptxas-verify -arch=sm_90 %}
 
 target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v16:16:16-v32:32:32-v64:64:64-v128:128:128-n16:32:64"
 target triple = "nvptx64-unknown-unknown"
@@ -76,3 +76,4 @@ entry:
 declare void @llvm.nvvm.prefetch.tensormap.p0(ptr)
 declare void @llvm.nvvm.prefetch.tensormap.p4(ptr addrspace(4))
 declare void @llvm.nvvm.prefetch.tensormap.p101(ptr addrspace(101))
+

Copy link

github-actions bot commented Aug 7, 2025

⚠️ We detected that you are using a GitHub private e-mail address to contribute to the repo.
Please turn off Keep my email addresses private setting in your account.
See LLVM Developer Policy and LLVM Discourse for more information.

@durga4github durga4github changed the title [NVPTX][CI] Add prefetch-inferas-test.ll sm version [NVPTX][Test-only] Add proper sm-version to ptxas-verify in prefetch-inferas-test.ll Aug 7, 2025
@durga4github durga4github merged commit ad3196d into llvm:main Aug 7, 2025
9 checks passed
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.

4 participants