This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Remove unused expand_*_loop from stmt.c
- From: law at redhat dot com
- To: Steven Bosscher <stevenb at suse dot de>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 18 May 2004 21:46:15 -0600
- Subject: Re: Remove unused expand_*_loop from stmt.c
- Reply-to: law at redhat dot com
In message <200405190031.11099.stevenb@suse.de>, Steven Bosscher writes:
>
>--Boundary-00=_v6oqAciiLw05dRt
>Content-Type: text/plain;
> charset="us-ascii"
>Content-Transfer-Encoding: 7bit
>Content-Disposition: inline
>
>These functions are now completely unused since I removed
>the final uses from the s390 backend.
>
>Bootstrapped & regtested on x86_64-unknown-linux-gnu, OK?
Yes.
>What are the rules for this kind of patch? There are other
>parts in stmt.c and expr.c that can easily be cleaned up
>now that we already linearize the tree while lowering GIMPLE.
>Most of these changes seem pretty obvious, they just remove
>unused functions. But since these diffs are quite large
>(like, 3 files changed, 6 (+), 400 (-), 50 (!) for this
>patch...) they're not "obviously obvious"... Thoughts?
Run them by the patch list first. I suspect 99.9% of the time you'd
get a quick response :-)
jeff