On 7/26/22 11:00, Taylor Simpson wrote:
So, instead of putting those in CFLAGS, put them in EXTRA_CFLAGS.

--- a/tests/tcg/hexagon/Makefile.target
+++ b/tests/tcg/hexagon/Makefile.target
@@ -20,6 +20,7 @@ EXTRA_RUNS =
CFLAGS += -Wno-incompatible-pointer-types -Wno-undefined-internal
  CFLAGS += -fno-unroll-loops
+EXTRA_CFLAGS += -mv67 -O2
HEX_SRC=$(SRC_PATH)/tests/tcg/hexagon
  VPATH += $(HEX_SRC)

Ah, ok.

Do I understand correctly that putting the flags in Makefile.target is the 
proper way and cross_cc_cflags is obsolete?

cross_cc_flags is intended to handle using one compiler for multiple targets, e.g. arm vs armbe.

Which is not what you're attempting to do; you're trying to test a particular isa. Compare tests/tcg/aarch64/Makefile.target:

bti-1 bti-3: CFLAGS += -mbranch-protection=standard

pauth-%: CFLAGS += -march=armv8.3-a

mte-%: CFLAGS += -march=armv8.5-a+memtag


where we set specific isa extensions for specific tests.


r~

Reply via email to