> -----Original Message----- > From: Taylor Simpson > Sent: Wednesday, February 8, 2023 9:19 AM > To: a...@rev.ng; qemu-devel@nongnu.org > Cc: richard.hender...@linaro.org; phi...@linaro.org; a...@rev.ng; Brian Cain > <bc...@quicinc.com>; Matheus Bernardino (QUIC) > <quic_mathb...@quicinc.com> > Subject: RE: [PATCH v5 10/14] Hexagon (tests/tcg/hexagon) Enable HVX tests > > > > > -----Original Message----- > > From: Anton Johansson <a...@rev.ng> > > Sent: Wednesday, February 8, 2023 6:54 AM > > To: Taylor Simpson <tsimp...@quicinc.com>; qemu-devel@nongnu.org > > Cc: richard.hender...@linaro.org; phi...@linaro.org; a...@rev.ng; Brian > Cain > > <bc...@quicinc.com>; Matheus Bernardino (QUIC) > > <quic_mathb...@quicinc.com> > > Subject: Re: [PATCH v5 10/14] Hexagon (tests/tcg/hexagon) Enable HVX > tests > > > > WARNING: This email originated from outside of Qualcomm. Please be > wary > > of any links or attachments, and do not enable macros. > > > > On 1/31/23 23:56, Taylor Simpson wrote: > > > +HEX_TESTS += hvx_histogram > > > > > > HEX_TESTS += test_abs > > > HEX_TESTS += test_bitcnt > > > @@ -78,3 +82,10 @@ TESTS += $(HEX_TESTS) > > > usr: usr.c > > > $(CC) $(CFLAGS) -mv67t -O2 -Wno-inline-asm > > > -Wno-expansion-to-defined $< -o $@ $(LDFLAGS) > > > > > > +scatter_gather: CFLAGS += -mhvx > > > +vector_add_int: CFLAGS += -mhvx -fvectorize > > > +hvx_misc: CFLAGS += -mhvx > > > +hvx_histogram: CFLAGS += -mhvx -Wno-gnu-folding-constant > > > + > > > +hvx_histogram: hvx_histogram.c hvx_histogram_row.S > > > + $(CC) $(CFLAGS) $(CROSS_CC_GUEST_CFLAGS) $^ -o $@ > > > > I am not able to run check-tcg locally, hvx_histogram fails due to missing > > ld- > > musl-hexagon > > > > TEST hvx_histogram on hexagon > > qemu-hexagon: Could not open '/lib/ld-musl-hexagon.so.1': No such file > or > > directory > > > > -- > > Anton Johansson, > > rev.ng Labs Srl. > > Strange. These are supposed to build statically. I'll investigate.
I need to add $(LDFLAGS) to the build command. -- a/tests/tcg/hexagon/Makefile.target +++ b/tests/tcg/hexagon/Makefile.target @@ -88,4 +88,4 @@ hvx_misc: CFLAGS += -mhvx hvx_histogram: CFLAGS += -mhvx -Wno-gnu-folding-constant hvx_histogram: hvx_histogram.c hvx_histogram_row.S - $(CC) $(CFLAGS) $(CROSS_CC_GUEST_CFLAGS) $^ -o $@ + $(CC) $(CFLAGS) $(CROSS_CC_GUEST_CFLAGS) $^ -o $@ $(LDFLAGS) I'll include this change in the next rev of the patch series. Thanks, Taylor
