[plugin] Mudflap plugin

Rafael Espindola espindola@google.com
Thu Jun 18 10:17:00 GMT 2009


> This is great work, as I said.  I think the first steps should be to
> separate the infrastructure changes and submit them separately.

I agree!

Having mudflap as a plugin would make sure it uses a cleaner interface
to the rest of the compiler.

Some items that I think can go in first

* ) I see that you added a flag_no_builtin_alloca. If Paolo's
suggestion of calling gimple_call_set_cannot_inline doesn't work, you
can use this flag in expand_builtin_alloca instead of flag_plugin.

* ) You moved the call to initialize_plugins to compile_file. We need
to load plugins early so that things like "gcc --help -v" can display
the plugins's help. Can you add a new callback instead of moving the
call to initialize_plugins?

* ) The move of PLUGIN_FINISH_UNIT looks correct to me. Paolo, why do
you need another callback?

>
> Paolo
>

Cheers,
-- 
Rafael Avila de Espindola

Google | Gordon House | Barrow Street | Dublin 4 | Ireland
Registered in Dublin, Ireland | Registration Number: 368047



More information about the Gcc-patches mailing list