This is the mail archive of the
fortran@gcc.gnu.org
mailing list for the GNU Fortran project.
Re: [PATCH] Fix Fortran DO loop fallback
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Martin Liška <mliska at suse dot cz>
- Cc: Jeff Law <law at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>
- Date: Wed, 13 Jul 2016 12:32:06 +0200
- Subject: Re: [PATCH] Fix Fortran DO loop fallback
- Authentication-results: sourceware.org; auth=none
- References: <cover.1467883947.git.mliska@suse.cz> <86a3b430-0282-305b-8796-5a696d53b46a@suse.cz> <9b6e6e1f-b62c-7ea4-1f72-417472fa96e4@redhat.com> <CAFiYyc1sotqJVzpNaKiZ4azVsWxeouvZSRvMGXP98XStQrLDoA@mail.gmail.com> <e9345442-52c2-b0c3-7c25-4073b92fc027@suse.cz> <CAFiYyc3Vydg63j5m0DN+r1SRKT9+qPyqCdTnXKy3AyTXcyo42w@mail.gmail.com> <f4d7f51c-8e71-7ada-fb63-73934646006d@suse.cz>
On Tue, Jul 12, 2016 at 4:54 PM, Martin Liška <mliska@suse.cz> wrote:
> On 07/12/2016 03:15 PM, Richard Biener wrote:
>> The scan for 1 *n_ after FRE looks still useful. Btw, the testcase
>> doesn't fail for me,
>> we _do_ hoist the division in PRE, just not with -m32 anymore. Can
>> you confirm this?
>
> Yeah, it works for both -m32 and -m64.
> This is final version of the patch, may I install it?
Changelog has a swapped entry.
Ok with fixing that.
Richard.
> Thanks,
> Martin