This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch] Fix to reload1.c's reload_cse_move2add.
- To: Toon Moene <toon at moene dot indiv dot nluug dot nl>
- Subject: Re: [Patch] Fix to reload1.c's reload_cse_move2add.
- From: Geoff Keating <geoffk at geoffk dot org>
- Date: 09 Jul 2001 13:29:42 -0700
- CC: gcc-patches at gcc dot gnu dot org
- References: <3B4A0DF7.F8B7E721@moene.indiv.nluug.nl>
Toon Moene <toon@moene.indiv.nluug.nl> writes:
> This is a multi-part message in MIME format.
> --------------CBD65685E6FEB544448CD6A1
> Content-Type: text/plain; charset=us-ascii
> Content-Transfer-Encoding: 7bit
>
> Not as much fun as the LO_SUM discussion, but a fix for a problem that's
> just as much wrong.
But has it been silently wrong for 11 years? :-)
> 2001-07-09 Toon Moene <toon@moene.indiv.nluug.nl>
>
> * expr.h: Adjust prototypes for have_add2_insn, have_sub2_insn.
> * optabs.c (have_add2_insn): Check whether the add insn chosen
> really accepts the operands. (have_sub2_insn): Ditto for sub insn.
> * reload1.c (reload_cse_move2add): Adjust calls of have_add2_insn.
You didn't say on which platform you bootstrapped it.
Are there really no users of have_sub2_insn? That seems odd... Oh well.
Assuming it passes bootstrap & test on some platform, this is OK.
--
- Geoffrey Keating <geoffk@geoffk.org>