> gen_expand and gen_split have some fairly complex tests that really just
> repeat what's in classify_insn.  This patch moves classify_insn to rtl.c,
> so that it's available to the generators.  It also adds a generator-specific
> UNKNOWN return code for cases where we need to choose at (GCC) runtime.
> 
> This is preparing for a patch series that I'll post about soon.
> 
> Bootstrapped & regression-tested on x86_64-linux-gnu.  Also tested
> with config-list.mk.  OK for the rtl bits?

Sure.

> The test for whether a barrier is needed seems a little weak: what
> about indirect jumps, jumps with match_operand sources, or jumps in
> PARALLELs?  That for another day though.  At least this patch means
> we'll only have to update one place rather than two.

Agreed.

-- 
Eric Botcazou

Reply via email to