This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: gcse.c and flow patch
- To: Andrew Macleod <amacleod at cygnus dot com>
- Subject: Re: gcse.c and flow patch
- From: Geoff Keating <geoffk at geoffk dot org>
- Date: 05 Apr 2001 15:41:43 -0700
- CC: gcc-patches at gcc dot gnu dot org
- References: <200104040239.TAA19910@rtl.cygnus.com>
Andrew Macleod <amacleod@cygnus.com> writes:
> If gcse tries to replace and expression with more than a single insn,
> we dont get the block information set properly, so add a new routine to
> do it correctly. Furthermore, set_block_num () is a deprecated function
> in flow.c that only gcse.c is using, so abandon it as well.
>
> A couple of these changes build on top of the previous load/store
> motion patch. If applied without that patch, a few lines would be
> rejected, but thats ok.
This is OK.
> Bootstrapped and verified on x86 linux.
>
> Andrew
>
>
> 2001-04-03 Andrew MacLeod <amacleod@redhat.com>
>
> * basic-block.h (set_new_block_for_insns): New Prototype.
> (set_block_num): Delete prototype.
> * flow.c (set_block_num): Remove obsolete function.
> (set_block_for_new_insns): Set BB for single or multiple insns.
> * gcse.c (handle_avail_expr): Use set_block_for_new_insns.
> (process_insn_end_bb): Use set_block_for_new_insns or
> set_block_for_insn instead of set_block_num.
> (pre_insert_copy_insn): Use set_block_for_new_insns.
> (update_ld_motion_stores): Use set_block_for_new_insns.
> (insert_insn_start_bb): Use set_block_for_new_insns.
> (replace_store_insn): Use set_block_for_new_insns.
...
--
- Geoffrey Keating <geoffk@geoffk.org>