Skip to content

[AMDGPU] Skip -mattr=dumpcode test on big-endian hosts #127064

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

Closed
wants to merge 1 commit into from

Conversation

jayfoad
Copy link
Contributor

@jayfoad jayfoad commented Feb 13, 2025

I did not know that the output is different depending on the host
endianness. This should be fixed properly but in the mean time this
patch fixes the buildbot failures.

I did not know that the output is different depending on the host
endianness. This should be fixed properly but in the mean time this
patch fixes the buildbot failures.
@llvmbot
Copy link
Member

llvmbot commented Feb 13, 2025

@llvm/pr-subscribers-backend-amdgpu

Author: Jay Foad (jayfoad)

Changes

I did not know that the output is different depending on the host
endianness. This should be fixed properly but in the mean time this
patch fixes the buildbot failures.


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/AMDGPU/dumpcode.ll (+1)
diff --git a/llvm/test/CodeGen/AMDGPU/dumpcode.ll b/llvm/test/CodeGen/AMDGPU/dumpcode.ll
index 1acec2997aa8f..27205cb644436 100644
--- a/llvm/test/CodeGen/AMDGPU/dumpcode.ll
+++ b/llvm/test/CodeGen/AMDGPU/dumpcode.ll
@@ -1,4 +1,5 @@
 ; RUN: llc -mtriple=amdgcn -mcpu=gfx1010 -mattr=dumpcode -filetype=obj < %s | llvm-objcopy --dump-section .AMDGPU.disasm=- - /dev/null | FileCheck %s -check-prefix=GFX10
+; REQUIRES: host-byteorder-little-endian
 
 ; GFX10: f:
 ; GFX10-NEXT: BB0_0:

@jayfoad
Copy link
Contributor Author

jayfoad commented Feb 13, 2025

Closing in favour of #127073.

@jayfoad jayfoad closed this Feb 13, 2025
@jayfoad jayfoad deleted the dumpcode-endianness branch February 13, 2025 16:45
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