[PATCH]: New warning option -Walloca

Manuel López-Ibáñez lopezibanez@gmail.com
Wed Sep 19 00:35:00 GMT 2007


> Keeping warn_disallowed_function_p() in opts.c (instead of
> c-common.c), and handling this new option in opts.c (instead of
> c-opts.c) makes the code a lot cleaner. This is because all this new
> code shares some common definitions (DEF_VEC_P of char_p), and
> functions (the refactored add_comma_separated_to_vector() function)
> with the -finstrument-functions-exclude* handling code in opts.c
>
> But if you feel strongly about moving warn_disallowed_function_p() to
> c-common.c, and handling the option in c-opts.c, I will make the
> changes you suggested.
>

I see now why it could be less cleaner and I don't have a good
solution for that. Then, you shouldn't move the option definition to
c.opt either.

Cheers,

Manuel.



More information about the Gcc-patches mailing list