[Patch] PR55189 enable -Wreturn-type by default

Chung-Ju Wu jasonwucj@gmail.com
Fri Dec 27 05:26:00 GMT 2013


2013/12/21 Sylvestre Ledru <sylvestre@debian.org>:
> Hello
>
> Following this thread http://gcc.gnu.org/ml/gcc/2013-11/msg00260.html
> and this bug,
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55189
>
> I would like to propose the two following patches:
>
> I am activating -Wreturn-type by defaut and add the option -Wmissing-return
>
[snip]
>
> Index: gcc/ChangeLog
> ===================================================================
> --- gcc/ChangeLog       (révision 206154)
> +++ gcc/ChangeLog       (copie de travail)
> @@ -1,3 +1,11 @@
> +2013-12-20  Sylvestre Ledru  <sylvestre@debian.org>
> +
> +        PR target/55189
> +        * -Wreturn-type enabled by default.
> +       * Introduce back the option -Wmissing-return (enabled by -Wall)
> +       It was included by default with -Wreturn-type
> +       * Update all tests failing because of these changes.
> +
>  2013-12-20  Eric Botcazou  <ebotcazou@adacore.com>
>         * config/arm/arm.c (arm_expand_prologue): In a nested APCS frame with

Hi, Sylvestre,

Sorry I have no right to approve this patch.
But I notice your ChangeLog formatting is not correct.

You can refer to other entries in ChangeLog to refine yours,
and then resubmit the patch for review. :)


Best regards,
jasonwucj



More information about the Gcc-patches mailing list