On Fri, Apr 01, 2022 at 10:00:37PM -0700, Palmer Dabbelt wrote: > GCC's semantics for "-march=X -march=Y" are that Y entirely overrides X, > but sparse takes the union of these two ISA strings. OK, but then I prefer to explicitly clear riscv_flags at the start of the function, like: diff --git a/target-riscv.c b/target-riscv.c index 3bba7c15ff1f..ca38d76e6465 100644 --- a/target-riscv.c +++ b/target-riscv.c @@ -54,6 +54,9 @@ static void parse_march_riscv(const char *arg) }; int i; + // Each -march=.. options entirely overrides previous ones + riscv_flags = 0; + for (i = 0; i < ARRAY_SIZE(basic_sets); i++) { const char *pat = basic_sets[i].pattern; size_t len = strlen(pat); -- Luc