RE: [PATCH v3] Hexagon: implement machine flag check

2024-04-04 Thread Andrew Pinski
> -Original Message- > From: Matheus Bernardino (QUIC) > Sent: Thursday, April 4, 2024 12:57 PM > To: m...@klomp.org > Cc: Brian Cain ; elfutils-devel@sourceware.org; Andrew > Pinski (QUIC) ; Matheus Bernardino (QUIC) > ; Sid Manning > Subject: Re: [PATCH v3] Hexagon: implement machine

Re: [PATCH v3] Hexagon: implement machine flag check

2024-04-04 Thread Matheus Tavares Bernardino
On Thu, Apr 04, 2024 at 21:43:11 +0200, Mark Wielaard wrote: > > > Note that we also have a public-inbox instance, which is useful in > some cases since you can address patches by message-id: >

Re: [PATCH v3] Hexagon: implement machine flag check

2024-04-04 Thread Mark Wielaard
Hi Matheus, On Thu, Apr 04, 2024 at 02:19:40PM -0300, Matheus Tavares Bernardino wrote: > This fixes the "invalid machine flag" error from eu-elflint when passing > hexagon binaries. > > * backends/hexagon_init.c (hexagon_init): Hook > machine_flag_check > *

[PATCH v3] Hexagon: implement machine flag check

2024-04-04 Thread Matheus Tavares Bernardino
This fixes the "invalid machine flag" error from eu-elflint when passing hexagon binaries. * backends/hexagon_init.c (hexagon_init): Hook machine_flag_check * backends/hexagon_symbol.c (hexagon_machine_flag_check): new function * libelf/elf-knowledge.h: add

Re: [PATCH v2 2/2] Hexagon: implement machine flag check

2024-04-04 Thread Mark Wielaard
Hi Brian, On Thu, 2024-04-04 at 16:31 +, Brian Cain wrote: > > > ... implies a new EF_HEXAGON_TINY 0x8000 definition BTW. > > > > You obviously know this architecture better than me. But is the TINY > > flag (bit 15) appropriate for all machines? It looks like it can only > > be set on V67

Re: [PATCH v2 1/2] Add support for Hexagon

2024-04-04 Thread Mark Wielaard
Hi Matheus, On Tue, 2024-04-02 at 16:55 -0300, Matheus Tavares Bernardino wrote: > This implements initial support for the Hexagon architecture. The > Hexagon ABI spec can be seen at > https://lists.llvm.org/pipermail/llvm-dev/attachments/20190916/21516a52/attachment-0001.pdf > > A

RE: [PATCH v2 2/2] Hexagon: implement machine flag check

2024-04-04 Thread Brian Cain
> -Original Message- > From: Mark Wielaard > Sent: Thursday, April 4, 2024 11:29 AM > To: Brian Cain ; Matheus Bernardino (QUIC) > ; elfutils-devel@sourceware.org > Cc: Sid Manning ; Andrew Pinski (QUIC) > > Subject: Re: [PATCH v2 2/2] Hexagon: implement machine flag check > >

Re: [PATCH v2 2/2] Hexagon: implement machine flag check

2024-04-04 Thread Mark Wielaard
Hi, On Tue, 2024-04-02 at 21:38 +, Brian Cain wrote: > > diff --git a/backends/hexagon_symbol.c b/backends/hexagon_symbol.c > > index b341243e..1e681e9f 100644 > > --- a/backends/hexagon_symbol.c > > +++ b/backends/hexagon_symbol.c > > @@ -56,3 +56,11 @@ hexagon_reloc_simple_type (Ebl *ebl