This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: one more version of the patch for PR61360
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: Vladimir Makarov <vmakarov at redhat dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Sandiford <rdsandiford at googlemail dot com>, "Gopalasubramanian, Ganesh" <Ganesh dot Gopalasubramanian at amd dot com>
- Date: Sun, 28 Sep 2014 21:54:16 +0200
- Subject: Re: RFA: one more version of the patch for PR61360
- Authentication-results: sourceware.org; auth=none
- References: <5425CD2E dot 8030708 at redhat dot com>
On Fri, Sep 26, 2014 at 10:31 PM, Vladimir Makarov <vmakarov@redhat.com> wrote:
> 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 <vmakarov@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.
Please also mention that this patch reverts:
2014-04-01 Richard Henderson <rth@redhat.com>
PR target/60704
* config/i386/i386.md (*float<SWI48><MODEF>2_sse): Leave the second
alternative enabled before register allocation.
The x86 part is OK.
BTW: I think that this patch should also be backported to 4.9 branch,
since the original patch and the PR60704 fix were also installed
there.
Thanks,
Uros.