Hi Vlad, Vladimir Makarov <vmaka...@redhat.com> writes: > I guess we achieved the consensus about the following patch to fix PR61360 > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360 > > The patch was successfully bootstrapped and tested (w/wo > -march=amdfam10) on x86/x86-64. > > Is it ok to commit to trunk? > > 2014-09-26 Vladimir Makarov <vmaka...@redhat.com> > > PR target/61360 > * lra.c (lra): Remove call of recog_init. > * recog.c (constrain_operands): Permit reg for memory constraint > when LRA is used. > * config/i386/i386.md (*float<SWI48:mode><MODEF:mode>2_sse): > Enable first alternative independently on RA stage.
Many thanks for doing this! I hadn't realised when seeing the bug originally that LRA made the recog.c part possible. Obviously I can't approve it, but this approach seems much cleaner to me. Richard