Skip to content

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