This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch][RFA] Remove compute_sets and CUIDs from gcse.c, and use DF instead
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Steven Bosscher <stevenb dot gcc at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 14 Apr 2009 10:31:27 +0200
- Subject: Re: [patch][RFA] Remove compute_sets and CUIDs from gcse.c, and use DF instead
- References: <571f6b510904131005tb48c495uf5216f21fed48c26@mail.gmail.com>
On Mon, Apr 13, 2009 at 7:05 PM, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
> Hi,
>
> This patch is the first step of DF-ifying gcse.c. ?Pretty straight-forward:
>
> * replace CUIDs with DF_INSN_LUIDs
> * replace compute_sets with the existing DF insn operand caches
>
> Bootstrapped&tested on ia64-unknown-linux-gnu.
> OK for trunk?
Ok without the ??? STEVEN ??? in
- /* ??? When we allocate this at the start of the function,
- the comment says that "this data is kept accurate during
- each pass". Apparently this is not so? FIXME. */
- free_reg_set_mem ();
- alloc_reg_set_mem (max_reg_num ());
- compute_sets ();
+ df_analyze (); /* ??? STEVEN ??? */
;)
Thanks,
Richard.