On Wed, Feb 5, 2014 at 8:27 AM, Alan Modra <amo...@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
