This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix bug in vect dumping code causing infinite loop
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Teresa Johnson <tejohnson at google dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, David Li <davidxl at google dot com>
- Date: Mon, 3 Feb 2014 16:44:46 +0100
- Subject: Re: [PATCH] Fix bug in vect dumping code causing infinite loop
- Authentication-results: sourceware.org; auth=none
- References: <CAAe5K+U51T57y5RKt6b0oUqo8hieXCgZXti1yTW65VUEUh6Cbw at mail dot gmail dot com>
On Mon, Feb 3, 2014 at 4:29 PM, Teresa Johnson <tejohnson@google.com> wrote:
> This patch fixes a bug in the dumping code, whereby an inner loop
> index variable was the same as the outer loop index variable, and the
> redef caused an infinite loop.
>
> Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk?
Ok
Thanks,
Richard.
> 2014-02-03 Teresa Johnson <tejohnson@google.com>
>
> * tree-vect-slp.c (vect_supported_load_permutation_p): Avoid
> redef of outer loop index variable.
>
> Index: tree-vect-slp.c
> ===================================================================
> --- tree-vect-slp.c (revision 207344)
> +++ tree-vect-slp.c (working copy)
> @@ -1103,8 +1103,8 @@ vect_supported_load_permutation_p (slp_instance sl
> FOR_EACH_VEC_ELT (node->load_permutation, j, next)
> dump_printf (MSG_NOTE, "%d ", next);
> else
> - for (i = 0; i < group_size; ++i)
> - dump_printf (MSG_NOTE, "%d ", i);
> + for (k = 0; k < group_size; ++k)
> + dump_printf (MSG_NOTE, "%d ", k);
> dump_printf (MSG_NOTE, "\n");
> }
>
>
> --
> Teresa Johnson | Software Engineer | tejohnson@google.com | 408-460-2413