This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Bug in reload_cse_move2add()
- To: Alexandre Oliva <aoliva at redhat dot com>
- Subject: Re: Bug in reload_cse_move2add()
- From: Richard Earnshaw <rearnsha at arm dot com>
- Date: Wed, 17 Jan 2001 15:15:24 +0000
- cc: Joern Rennecke <amylaar at redhat dot com>, rearnsha at arm dot com, rearnsha at buzzard dot freeserve dot co dot uk (Richard Earnshaw), gcc-patches at gcc dot gnu dot org, law at redhat dot com
- Organization: ARM Ltd.
- Reply-To: rearnsha at arm dot com
> On Jan 17, 2001, Joern Rennecke <amylaar@redhat.com> wrote:
>
> > You ignore the problem of multi-hard-register stores.
>
> Do I? It seems to me that it should work just fine. Not only the
> luid is used, but also the reg number. So multiple sets in the same
> insn should work. What am I missing?
Well the good news is that arm-netbsd has bootstrapped with your latest
version. However, what about the conditional execution issue:
note_stores just strips the cond_exec part off, so we may falsely set a
base if we do not handle it explicitly ourselves. I think we could handle
it by manually striping that part off and passing it as the data operand
to move2add_note_stores; then, if data was non-null, we would always set
the register as unknown if it was modified.
R.