Skip to content

Conversation

@AlexMaclean
Copy link
Member

Use 'not' to confirm the test has the desired behavior and move it to passing.

@llvmbot
Copy link
Member

llvmbot commented Mar 11, 2025

@llvm/pr-subscribers-backend-nvptx

Author: Alex MacLean (AlexMaclean)

Changes

Use 'not' to confirm the test has the desired behavior and move it to passing.


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/NVPTX/gvar-init.ll (+2-2)
diff --git a/llvm/test/CodeGen/NVPTX/gvar-init.ll b/llvm/test/CodeGen/NVPTX/gvar-init.ll
index 6fd650f7f326c..2c7963fd0a4b5 100644
--- a/llvm/test/CodeGen/NVPTX/gvar-init.ll
+++ b/llvm/test/CodeGen/NVPTX/gvar-init.ll
@@ -1,5 +1,5 @@
-; RUN: llc < %s -mtriple=nvptx -mcpu=sm_20 | FileCheck %s
+; RUN: not --crash llc < %s -mtriple=nvptx -mcpu=sm_20 2>&1 | FileCheck %s
 
 ; Error out if initializer is given for address spaces that do not support initializers
-; XFAIL: *
+; CHECK: LLVM ERROR: initial value of 'g0' is not allowed in addrspace(3)
 @g0 = addrspace(3) global i32 42

@AlexMaclean AlexMaclean merged commit 48b1991 into llvm:main Mar 12, 2025
13 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.

3 participants