This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [RS6000] SDmode and reload
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>, Alan Modra <amodra at gmail dot com>
- Date: Wed, 5 Feb 2014 10:32:27 -0500
- Subject: Re: [RS6000] SDmode and reload
- Authentication-results: sourceware.org; auth=none
- References: <20140205132747 dot GC3596 at bubble dot grove dot modra dot org>
On Wed, Feb 5, 2014 at 8:27 AM, Alan Modra <amodra@gmail.com> wrote:
> Vlad added rs6000_secondary_memory_needed_mode for lra, and in so
> doing broke reload's use of cfun->machine->sdmode_stack_slot in
> rs6000_secondary_memory_needed_rtx (mode was no longer SDmode there).
> Bootstrapped and regression tested powerpc64-linux, both with and
> without -mlra. OK to commit?
>
> PR target/60032
> * config/rs6000/rs6000.c (rs6000_secondary_memory_needed_mode): Only
> change SDmode to DDmode when lra_in_progress.
>
> * gcc.target/powerpc/pr60032.c: New.
So Vlad added the definition to the rs6000 port, but it only is needed
by LRA and does not interact correctly with classic reload.
Okay.
- David