On Fri, 27 May 2022 at 19:11, Richard Henderson <[email protected]> wrote: > > Put the inline function near the array declaration. > > Signed-off-by: Richard Henderson <[email protected]> > --- > target/arm/vec_internal.h | 8 +++++++- > target/arm/sve_helper.c | 9 --------- > 2 files changed, 7 insertions(+), 10 deletions(-) > > diff --git a/target/arm/vec_internal.h b/target/arm/vec_internal.h > index 1d63402042..d1a1ea4a66 100644 > --- a/target/arm/vec_internal.h > +++ b/target/arm/vec_internal.h > @@ -50,8 +50,14 @@ > #define H8(x) (x) > #define H1_8(x) (x) > > -/* Data for expanding active predicate bits to bytes, for byte elements. */ > +/* > + * Expand active predicate bits to bytes, for byte elements. > + */ > extern const uint64_t expand_pred_b_data[256]; > +static inline uint64_t expand_pred_b(uint8_t byte) > +{ > + return expand_pred_b_data[byte]; > +}
Reviewed-by: Peter Maydell <[email protected]> thanks -- PMM
