Re: [PATCH][i386]Fix PR 57756

[Sriraman Tallam]

On Tue, Oct 15, 2013 at 10:54 PM, Alan Modra <amo...@gmail.com> wrote:
> On Tue, Oct 15, 2013 at 02:45:23PM -0700, Sriraman Tallam wrote:
>> I committed this patch after making the above change.
>
> /src/gcc-virgin/gcc/config/rs6000/rs6000.c: At global scope:
> /src/gcc-virgin/gcc/config/rs6000/rs6000.c:31122:29: error: invalid 
> conversion from ‘void (*)(cl_target_option*)’ to ‘void (*)(cl_target_option*, 
> gcc_options*)’ [-fpermissive]
> /src/gcc-virgin/gcc/config/rs6000/rs6000.c:31122:29: error: invalid 
> conversion from ‘void (*)(cl_target_option*)’ to ‘void (*)(gcc_options*, 
> cl_target_option*)’ [-fpermissive]

This patch fixes it, ok to submit?


* config/rs6000/rs6000.c (rs6000_function_specific_save): New unused parameter.
(rs6000_function_specifc_restore): Ditto.

Index: config/rs6000/rs6000.c
===================================================================
--- config/rs6000/rs6000.c      (revision 203711)
+++ config/rs6000/rs6000.c      (working copy)
@@ -29995,7 +29995,8 @@ rs6000_set_current_function (tree fndecl)
 /* Save the current options */

 static void
-rs6000_function_specific_save (struct cl_target_option *ptr)
+rs6000_function_specific_save (struct cl_target_option *ptr,
+                              struct gcc_options *ARG_UNUSED (opts))
 {
   ptr->x_rs6000_isa_flags = rs6000_isa_flags;
   ptr->x_rs6000_isa_flags_explicit = rs6000_isa_flags_explicit;
@@ -30004,7 +30005,8 @@ static void
 /* Restore the current options */

 static void
-rs6000_function_specific_restore (struct cl_target_option *ptr)
+rs6000_function_specific_restore (struct gcc_options *ARG_UNUSED (opts),
+                                 struct cl_target_option *ptr)
 {
   rs6000_isa_flags = ptr->x_rs6000_isa_flags;
   rs6000_isa_flags_explicit = ptr->x_rs6000_isa_flags_explicit;


Thanks
Sri


>
>
> --
> Alan Modra
> Australia Development Lab, IBM