This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PR87469] ICE in record_estimate, at tree-ssa-loop-niter.c
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Kugan Vivekanandarajah <kugan dot vivekanandarajah at linaro dot org>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 29 Oct 2018 15:22:12 +0100
- Subject: Re: [PR87469] ICE in record_estimate, at tree-ssa-loop-niter.c
- References: <CAELXzTMjZzcjLiFDz-3nWsbUrA--N5kjuM8Mu7fGyHi9_m+0Kg@mail.gmail.com>
On Sun, Oct 28, 2018 at 1:11 AM Kugan Vivekanandarajah
<kugan.vivekanandarajah@linaro.org> wrote:
>
> Hi,
>
> In the testcase provided in the bug report, max value for niter
> estimation is off by one when it is INTEGER_CST. As a results it
> asserts at the place where it is checked for equality.
> Attached patch fixes this. Bootstrapped and regression tested on
> x86_64-linux-gnu with no new regression. Is this OK?
OK.
> Thanks,
> Kugan
>
> gcc/testsuite/ChangeLog:
>
> 2018-10-26 Kugan Vivekanandarajah <kuganv@linaro.org>
>
> PR middle-end/87469
> * g++.dg/pr87469.C: New test.
>
> gcc/ChangeLog:
>
> 2018-10-26 Kugan Vivekanandarajah <kuganv@linaro.org>
>
> PR middle-end/87469
> * tree-ssa-loop-niter.c (number_of_iterations_popcount): Fix niter
> max value.