> -----Original Message-----
> From: Matheus Tavares Bernardino <[email protected]>
> Sent: Thursday, July 6, 2023 5:23 AM
> To: [email protected]
> Cc: [email protected]; [email protected];
> [email protected]; [email protected];
> [email protected]
> Subject: Re: [PATCH v2] Hexagon: move GETPC() calls to top level helpers
>
>
> > [email protected] wrote:
> >
> > > -----Original Message-----
> > > From: Matheus Tavares Bernardino <[email protected]>
> > > Sent: Wednesday, July 5, 2023 12:35 PM
> > > To: [email protected]
> > > Cc: [email protected]; [email protected];
> > > [email protected]; [email protected];
> > > [email protected]
> > > Subject: [PATCH v2] Hexagon: move GETPC() calls to top level helpers
> > >
> > > a/target/hexagon/op_helper.h b/target/hexagon/op_helper.h index
> > > 8f3764d15e..7744e819ef 100644
> > > --- a/target/hexagon/op_helper.h
> > > +++ b/target/hexagon/op_helper.h
> > > +void check_noshuf(CPUHexagonState *env, bool pkt_has_store_s1,
> > > + uint32_t slot, target_ulong vaddr, int size,
> > > +uintptr_t ra);
> >
> > Are you sure this needs to be non-static?
>
> Yeah, since we remove the mem_load*() functions, check_noshuf() must
> now be visible to the other compilation units that include macros.h, as we
will
> expand the fLOAD() macro to call it.
Since the generated helper functions are included at the bottom of
op_helper.c
#include "helper_funcs_generated.c.inc"
it can be static.
It needs to be guarded with
#ifndef CONFIG_HEXAGON_IDEF_PARSER
because it is not used when by any of the idef-parser functions.
Thanks,
Taylor
