This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: Make classify_insn available to the generators
- From: Eric Botcazou <ebotcazou at adacore dot com>
- To: Richard Sandiford <rdsandiford at googlemail dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Sat, 13 Jun 2015 12:25:14 +0200
- Subject: Re: RFA: Make classify_insn available to the generators
- Authentication-results: sourceware.org; auth=none
- References: <87381w3p88 dot fsf at googlemail dot com>
> 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