Skip to content

Conversation

paperchalice
Copy link
Contributor

The test will generate an empty regalloc-amdgpu.s file in test, which causes an unresolved test.

The test will generate an empty regalloc.s file in test, which causes an unresolved test.
@paperchalice paperchalice added the skip-precommit-approval PR for CI feedback, not intended for review label Jun 7, 2024
@paperchalice paperchalice changed the title [test] Dont' generate regalloc-amdgpu.s in #94426 [test] Don't generate regalloc-amdgpu.s in #94426 Jun 7, 2024
@paperchalice paperchalice merged commit a6350d6 into llvm:main Jun 7, 2024
@llvmbot
Copy link
Member

llvmbot commented Jun 7, 2024

@llvm/pr-subscribers-backend-amdgpu

Author: None (paperchalice)

Changes

The test will generate an empty regalloc-amdgpu.s file in test, which causes an unresolved test.


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

1 Files Affected:

  • (modified) llvm/test/tools/llc/new-pm/regalloc-amdgpu.mir (+1-1)
diff --git a/llvm/test/tools/llc/new-pm/regalloc-amdgpu.mir b/llvm/test/tools/llc/new-pm/regalloc-amdgpu.mir
index 872c35237c20a..0603ffc4c5ea2 100644
--- a/llvm/test/tools/llc/new-pm/regalloc-amdgpu.mir
+++ b/llvm/test/tools/llc/new-pm/regalloc-amdgpu.mir
@@ -1,6 +1,6 @@
 # REQUIRES: amdgpu-registered-target
 # RUN: llc -mtriple=amdgcn --passes='regallocfast<filter=sgpr>' --print-pipeline-passes %s | FileCheck %s --check-prefix=PASS
-# RUN: not llc -mtriple=amdgcn --passes='regallocfast<filter=bad-filter>' --print-pipeline-passes %s 2>&1 | FileCheck %s --check-prefix=BAD-FILTER
+# RUN: not llc -mtriple=amdgcn --passes='regallocfast<filter=bad-filter>' --print-pipeline-passes --filetype=null %s 2>&1 | FileCheck %s --check-prefix=BAD-FILTER
 
 # PASS: regallocfast<filter=sgpr>
 # BAD-FILTER: invalid regallocfast register filter 'bad-filter'

@paperchalice paperchalice deleted the rm-s branch June 7, 2024 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend:AMDGPU skip-precommit-approval PR for CI feedback, not intended for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants