[PATCH C/C++/middle-end] move Wunused-parameter to the FEs

[Jeff Law]

On 06/29/2015 09:48 AM, Manuel López-Ibáñez wrote:
> Wunused-parameter warnings are given from cgraph::finalize_function,
> which is the middle-end. This is an oddity compared to other
> -Wunused-* warnings. Moreover, Fortran has its own definition of
> -Wunused-parameter that conflicts with the middle-end definition.
>
> This patch moves the middle-end part of Wunused-parameter to the C/C++
> FEs. I'm not sure if other FEs expected this warning to work. If so,
> they do not seem to test for it. Ada, for example, explicitly disables
> it.
>
> This also fixes [fortran/66605] -Wunused-parameter causes internal
> compiler error with gfortran 5.1.0.
>
> Bootstrapped & regression tested on x86_64-linux-gnu with languages
> c,c++,objc,fortran,ada,obj-c++
>
> OK?
>
>
> gcc/ChangeLog:
>
> 2015-06-28  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/66605
>      * cgraphunit.c (cgraph_node::finalize_function): Do not call
>      do_warn_unused_parameter.
>      * function.c (do_warn_unused_parameter): Move from here.
>      * function.h (do_warn_unused_parameter): Do not declare.
>
> gcc/c-family/ChangeLog:
>
> 2015-06-28  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/66605
>     * c-common.c (do_warn_unused_parameter): Move here.
>      * c-common.h (do_warn_unused_parameter): Declare.
>
> gcc/ada/ChangeLog:
>
> 2015-06-28  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/66605
>     * gcc-interface/misc.c (gnat_post_options): No need to disable
>      warn_unused_parameter anymore.
>
> gcc/cp/ChangeLog:
>
> 2015-06-28  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/66605
>     * decl.c (finish_function): Call do_warn_unused_parameter.
>
> gcc/testsuite/ChangeLog:
>
> 2015-06-28  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/66605
>     * gfortran.dg/wunused-parameter.f90: New test.
>
> gcc/c/ChangeLog:
>
> 2015-06-28  Manuel López-Ibáñez  <manu@gcc.gnu.org>
>
>      PR fortran/66605
>     * c-decl.c (finish_function): Call do_warn_unused_parameter.
>
OK for the trunk after fixing the whitespace vs tabs issues in the 
ChangeLog entries.  Nitty, I know :-)

Jeff