[all-commits] [llvm/llvm-project] ce93e5: [AMDGPU] Add AMDGPU support for llvm-objcopy

[David Salinas via All-commits]

  Branch: refs/heads/users/dsalinas/amdgpu-llvm-objcopy
  Home:   https://github.com/llvm/llvm-project
  Commit: ce93e57257dbb93695a3609c3f574d37cfcf4e86
      https://github.com/llvm/llvm-project/commit/ce93e57257dbb93695a3609c3f574d37cfcf4e86
  Author: david-salinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/ELF/cross-arch-headers.test
    M llvm/tools/llvm-objcopy/ObjcopyOptions.cpp

  Log Message:
  -----------
  [AMDGPU] Add AMDGPU support for llvm-objcopy

Co-authored-by: Aakanksha Patil <paakan at amd.com>


  Commit: c304cdb609de308d4359fe2c6777eb5b252db880
      https://github.com/llvm/llvm-project/commit/c304cdb609de308d4359fe2c6777eb5b252db880
  Author: dsalinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/ELF/cross-arch-headers.test

  Log Message:
  -----------
  [llvm-objcopy] Fix AMDGPU arch string in test: amdgpu not amdgcn

llvm-readobj reports 'Arch: amdgpu' for EM_AMDGPU ELF files
(the generic AMDGPU ELF format used by elf64-amdgpu). The test
was incorrectly expecting 'amdgcn', which is the AMDGCN-specific
arch string used by ROCm HSA code objects.


  Commit: b72ec742cfd01d79fb5e9c5329517f8d3d94326f
      https://github.com/llvm/llvm-project/commit/b72ec742cfd01d79fb5e9c5329517f8d3d94326f
  Author: dsalinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/ELF/binary-output-target.test

  Log Message:
  -----------
  [llvm-objcopy] Add AMDGPU case to binary-output-target.test

Add test coverage for converting binary input to elf64-amdgpu format,
verifying the output has the correct format string, arch (amdgpu),
and machine type (EM_AMDGPU 0xE0). Follows the same pattern as all
other architectures in this test file.


  Commit: 3d06c44f69dab1cc7267b8f5921ea1c02dec51d3
      https://github.com/llvm/llvm-project/commit/3d06c44f69dab1cc7267b8f5921ea1c02dec51d3
  Author: dsalinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/ELF/binary-output-target.test

  Log Message:
  -----------
  [llvm-objcopy] Fix AMDGPU arch checks in tests

ELFObjectFile.h getArch() for EM_AMDGPU returns Triple::UnknownArch
when e_flags & EF_AMDGPU_MACH is 0 (no GPU target specified). Only
when a MACH flag in the AMDGCN range is present does it return
Triple::amdgpu.

- cross-arch-headers.test: restore EF_AMDGPU_MACH_AMDGCN_GFX900 flag
  on the input ELF so that after format conversion the output correctly
  reports Arch: amdgpu.

- binary-output-target.test: expect Arch: unknown since converting
  from raw binary input (-I binary) produces an ELF with e_flags=0
  (no MACH flags), giving UnknownArch. This is correct behavior.


  Commit: 9e4929c4523c69a0f89ab6bad4891292943427ea
      https://github.com/llvm/llvm-project/commit/9e4929c4523c69a0f89ab6bad4891292943427ea
  Author: dsalinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/ELF/cross-arch-headers.test

  Log Message:
  -----------
  [llvm-objcopy] Address review feedback for AMDGPU test in cross-arch-headers

Per reviewer feedback, use the existing non-AMDGPU input (%t.o, EM_NONE)
to test conversion to elf64-amdgpu. This properly demonstrates that
--output-format changes the machine type, consistent with all other cases
in this test file.

The output reports Arch: unknown because converting from a non-AMDGPU ELF
produces e_flags=0 (no EF_AMDGPU_MACH set); added a comment explaining
this. Flag control is a separate concern for a follow-on PR.


  Commit: 79f12635e19ce313e3ef40a9fcc500983ab0e8b1
      https://github.com/llvm/llvm-project/commit/79f12635e19ce313e3ef40a9fcc500983ab0e8b1
  Author: dsalinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/test/tools/llvm-objcopy/ELF/binary-output-target.test
    M llvm/test/tools/llvm-objcopy/ELF/cross-arch-headers.test

  Log Message:
  -----------
  [llvm-objcopy] Address reviewer feedback on AMDGPU test cleanups

- Remove unused -DMACHINE yaml2obj template variable in cross-arch-headers.test,
  hardcode Machine: EM_NONE directly in the YAML instead
- Remove unused Flags: [[FLAGS=<none>]] template variable in cross-arch-headers.test
- Add comment in binary-output-target.test explaining that Arch: unknown is
  intentional when converting from binary (e_flags=0, no EF_AMDGPU_MACH set)


  Commit: e1ecc647444d847ac26d67344fbf886490218bdb
      https://github.com/llvm/llvm-project/commit/e1ecc647444d847ac26d67344fbf886490218bdb
  Author: dsalinas <dsalinas at amd.com>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M llvm/docs/CommandGuide/llvm-objcopy.rst

  Log Message:
  -----------
  [llvm-objcopy] Add elf64-amdgpu to supported formats in command guide

Update the llvm-objcopy command guide's "Supported formats" section to
include elf64-amdgpu, added in the preceding commit.


Compare: https://github.com/llvm/llvm-project/compare/393e75abe89e...e1ecc647444d

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications