This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Fix nb_iterations calculation in tree-vect-loop-manip.c

From: Richard Biener <richard dot guenther at gmail dot com>
To: Richard Biener <richard dot guenther at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, richard dot sandiford at arm dot com
Date: Wed, 16 Nov 2016 14:02:06 +0100
Subject: Re: Fix nb_iterations calculation in tree-vect-loop-manip.c
Authentication-results: sourceware.org; auth=none
References: <8737is52us.fsf@e105548-lin.cambridge.arm.com> <CAFiYyc38q4tvH0v2B00vbU62nmOwZPfi9AgeeYhwi7J3P2k31g@mail.gmail.com> <87r36c3fi2.fsf@e105548-lin.cambridge.arm.com>

On Tue, Nov 15, 2016 at 4:54 PM, Richard Sandiford
<richard.sandiford@arm.com> wrote:
> Richard Biener <richard.guenther@gmail.com> writes:
>> On Tue, Nov 15, 2016 at 1:44 PM, Richard Sandiford
>> <richard.sandiford@arm.com> wrote:
>>> We previously stored the number of loop iterations rather
>>> than the number of latch iterations.
>>
>> So ->nb_iterations was unused without SVE?  Otherwise can you please
>> add a testcase?
>
> TBH I can't remember whether we noticed this by inspection or whether
> it did manifest in the output somehow.  If it did, it would have been
> an extra unnecessary iteration after complete unrolling, but usually
> a later patch would remove the iteration as dead.

Ok then.

Thanks,
Richard.

> Thanks,
> Richard

References:
- Fix nb_iterations calculation in tree-vect-loop-manip.c
  - From: Richard Sandiford
- Re: Fix nb_iterations calculation in tree-vect-loop-manip.c
  - From: Richard Biener
- Re: Fix nb_iterations calculation in tree-vect-loop-manip.c
  - From: Richard Sandiford

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]