Skip to content
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

[CodeGen] Disable ran-out-of-registers-error* tests #120142

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

thurstond
Copy link
Contributor

Two tests are failing on the buildbot in stage2/asan with a stack use-after-scope:
https://lab.llvm.org/buildbot/#/builders/52/builds/4533 (first failure here; contains #119492 and #119640)
...
https://lab.llvm.org/buildbot/#/builders/52/builds/4550

This patch disables the tests for now, to allow the bots to return to green (instead of reverting the patch series).

These two the tests are failing on the buildbot in stage2/asan with a
stack use-after-scope:
    https://lab.llvm.org/buildbot/#/builders/52/builds/4533 (first failure here; contains llvm#119492 and llvm#119640)
    ...
    https://lab.llvm.org/buildbot/#/builders/52/builds/4550

This patch disables the tests for now, to allow the bots to return to
green.
@thurstond thurstond added the skip-precommit-approval PR for CI feedback, not intended for review label Dec 16, 2024
@thurstond thurstond merged commit 1297933 into llvm:main Dec 16, 2024
7 of 9 checks passed
@llvmbot
Copy link
Member

llvmbot commented Dec 16, 2024

@llvm/pr-subscribers-backend-amdgpu

Author: Thurston Dang (thurstond)

Changes

Two tests are failing on the buildbot in stage2/asan with a stack use-after-scope:
https://lab.llvm.org/buildbot/#/builders/52/builds/4533 (first failure here; contains #119492 and #119640)
...
https://lab.llvm.org/buildbot/#/builders/52/builds/4550

This patch disables the tests for now, to allow the bots to return to green (instead of reverting the patch series).


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

2 Files Affected:

  • (modified) llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll (+1)
  • (modified) llvm/test/CodeGen/AMDGPU/ran-out-of-registers-errors.ll (+2)
diff --git a/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll b/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll
index 388a8e804a8896..03860b7f366661 100644
--- a/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll
+++ b/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-error-all-regs-reserved.ll
@@ -1,3 +1,4 @@
+; UNSUPPORTED: target={{.*}}
 ; RUN: not llc -mtriple=amdgcn-amd-amdhsa -mcpu=gfx908 -vgpr-regalloc=greedy -verify-machineinstrs=0 -filetype=null %s 2>&1 | FileCheck -implicit-check-not=error %s
 ; RUN: not llc -mtriple=amdgcn-amd-amdhsa -mcpu=gfx908 -vgpr-regalloc=basic -verify-machineinstrs=0 -filetype=null %s 2>&1 | FileCheck -implicit-check-not=error %s
 ; RUN: not llc -mtriple=amdgcn-amd-amdhsa -mcpu=gfx908 -vgpr-regalloc=fast -verify-machineinstrs=0 -filetype=null %s 2>&1 | FileCheck -implicit-check-not=error %s
diff --git a/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-errors.ll b/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-errors.ll
index bd1752d21507ca..da229d157fb763 100644
--- a/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-errors.ll
+++ b/llvm/test/CodeGen/AMDGPU/ran-out-of-registers-errors.ll
@@ -1,3 +1,5 @@
+; UNSUPPORTED: target={{.*}}
+;
 ; RUN: not llc -mtriple=amdgcn-amd-amdhsa -stress-regalloc=1 -vgpr-regalloc=greedy -filetype=null %s 2>&1 | FileCheck -check-prefixes=CHECK,GREEDY -implicit-check-not=error %s
 ; RUN: not llc -mtriple=amdgcn-amd-amdhsa -stress-regalloc=1 -vgpr-regalloc=basic -filetype=null %s 2>&1 | FileCheck -implicit-check-not=error -check-prefixes=CHECK,BASIC %s
 ; RUN: not llc -mtriple=amdgcn-amd-amdhsa -stress-regalloc=1 -vgpr-regalloc=fast -filetype=null %s 2>&1 | FileCheck -implicit-check-not=error -check-prefixes=CHECK,FAST %s

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