This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Regression caused by reload_cse_simplify_operands bugs
- From: Ian Lance Taylor <ian at airs dot com>
- To: Ulrich Weigand <uweigand at de dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: 10 Nov 2005 17:12:35 -0800
- Subject: Re: [PATCH] Regression caused by reload_cse_simplify_operands bugs
- References: <200511102322.jAANM0iE015498@53v30g15.boeblingen.de.ibm.com>
Ulrich Weigand <uweigand@de.ibm.com> writes:
> * postreload.c (reload_cse_simplify_operands): Fix bug in sorting
> algorithm so as to choose the best, not the worst, alternative.
> Reset accumulated register class before processing next alternative.
This is OK. Thanks.
I guess nobody noticed since it most be rather rare that there would
be two different alternatives available anyhow. It's particularly
hard to imagine it happening on a RISC chip.
Looking at the code, I wonder why it carefully sorts the entire
alternative_order array and then only uses the first element of it.
Ian