Re: PING [PATCH] enable -fprintf-return-value by default

[Sandra Loosemore <sandra at codesourcery dot com>]

On 11/16/2016 09:49 AM, Martin Sebor wrote:
> I'm looking for an approval of the attached patch.
>
> I've adjusted the documentation based on Sandra's input (i.e.,
> documented the negative of the option rather than the positive;
> thank you for the review, btw.)
>
> On 11/08/2016 08:13 PM, Martin Sebor wrote:
> > The -fprintf-return-value optimization has been disabled since
> > the last time it caused a bootstrap failure on powerpc64le.  With
> > the underlying problems fixed GCC has bootstrapped fine on all of
> > powerpc64, powerpc64le and x86_64 and tested with no regressions.
> > I'd like to re-enable the option.  The attached patch does that.
>
> [snip]
>
> Index: gcc/doc/invoke.texi
> ===================================================================
> --- gcc/doc/invoke.texi	(revision 242500)
> +++ gcc/doc/invoke.texi	(working copy)
> @@ -384,7 +384,7 @@ Objective-C and Objective-C++ Dialects}.
>  -fno-toplevel-reorder -fno-trapping-math -fno-zero-initialized-in-bss @gol
>  -fomit-frame-pointer -foptimize-sibling-calls @gol
>  -fpartial-inlining -fpeel-loops -fpredictive-commoning @gol
> --fprefetch-loop-arrays -fprintf-return-value @gol
> +-fprefetch-loop-arrays -fno-printf-return-value @gol
>  -fprofile-correction @gol
>  -fprofile-use -fprofile-use=@var{path} -fprofile-values @gol
>  -fprofile-reorder-functions @gol

Please keep this list alphabetized -- the other "-fno-*" options are
sorted as such.  The documentation parts of the patch are OK with that 
fixed, but I can't approve changing the default for the option.

-Sandra