mbox series

[v2,0/4] Add support for new AMDGPU architectures

Message ID 20260507194615.943267-1-simon.marchi@efficios.com
Headers
Series Add support for new AMDGPU architectures |

Message

Simon Marchi May 7, 2026, 7:43 p.m. UTC 
  This is v2 of:

https://inbox.sourceware.org/binutils/20251031175411.1303161-1-luis.machado@amd.com/

I address the review comments, and I also added the gfx1250 and
gfx12-5-generic architectures.

Patch 4 is new.


Lancelot Six (2):
  Update list of supported AMDGPU architectures
  readelf: Add support for AMDGPU code object V6

Simon Marchi (2):
  Add AMDGPU generic targets
  include: add reference URLs in elf/amdgpu.h

 bfd/archures.c       | 44 ++++++++++++++++++++++++++-------------
 bfd/bfd-in2.h        | 46 +++++++++++++++++++++++++++--------------
 bfd/cpu-amdgcn.c     | 34 ++++++++++++++++++++++--------
 binutils/readelf.c   | 31 +++++++++++++++++++++++++++-
 include/elf/amdgpu.h | 49 ++++++++++++++++++++++++++++++++++++++------
 5 files changed, 159 insertions(+), 45 deletions(-)


base-commit: cca4420639a1390f1fb23286618efbe494987825

Comments

Jan Beulich May 8, 2026, 8:23 a.m. UTC | #1 
On 07.05.2026 21:43, Simon Marchi wrote:
> This is v2 of:
> 
> https://inbox.sourceware.org/binutils/20251031175411.1303161-1-luis.machado@amd.com/
> 
> I address the review comments, and I also added the gfx1250 and
> gfx12-5-generic architectures.
> 
> Patch 4 is new.
> 
> 
> Lancelot Six (2):
>   Update list of supported AMDGPU architectures
>   readelf: Add support for AMDGPU code object V6
> 
> Simon Marchi (2):
>   Add AMDGPU generic targets
>   include: add reference URLs in elf/amdgpu.h
> 
>  bfd/archures.c       | 44 ++++++++++++++++++++++++++-------------
>  bfd/bfd-in2.h        | 46 +++++++++++++++++++++++++++--------------
>  bfd/cpu-amdgcn.c     | 34 ++++++++++++++++++++++--------
>  binutils/readelf.c   | 31 +++++++++++++++++++++++++++-
>  include/elf/amdgpu.h | 49 ++++++++++++++++++++++++++++++++++++++------
>  5 files changed, 159 insertions(+), 45 deletions(-)

Except for the one aspect in patch 1: Okay for the entire series.

Jan