This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch, libfortran] Fix part of PR 79956
- From: Jerry DeLisle <jvdelisle at charter dot net>
- To: Thomas Koenig <tkoenig at netcologne dot de>, "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 9 Mar 2017 14:26:55 -0800
- Subject: Re: [patch, libfortran] Fix part of PR 79956
- Authentication-results: sourceware.org; auth=none
- References: <4292df62-3785-3f27-33b4-e2814fd42b3b@netcologne.de>
On 03/09/2017 01:32 PM, Thomas Koenig wrote:
> Hello world,
>
> the attached patch fixes a part of PR 79956, where
> warnings about maybe uninitialized variables broke
> bootstrap-O3 due to -Wmaybe-uninitialized warnings.
>
> I have also attached a compressed version of the patch including the
> changes in generated/ in case anybody wants to test, but does not have
> --enable-maintainer-mode on their system. This version of the patch
> compresses very well indeed :-)
>
> Regression-tested. OK for trunk?
>
Yes, OK
with following nit.
On the last hunk of diff there is a misspelling in the existing comment:
> + if (n >= rank)
> {
> /* Break out of the look. */
I think the word look should be loop ?
Jerry