Re: [patch][plugin] Add PLUGIN_START_UNIT

[Olatunji Ruwase <tjruwase at google dot com>]

Patch attached with suggested testcase.

Thanks

tunji

On Thu, Jun 25, 2009 at 5:41 AM, Rafael Espindola<espindola@google.com> wrote:
> 2009/6/23 Olatunji Ruwase <tjruwase@google.com>:
>> Hello,
>> ?This patch adds PLUGIN_START_UNIT callbacks.
>>
>> ?PLUGIN_START_UNIT callbacks allow plugins to initialize state for
>> each translation unit, before it is compiled.
>> ?For example, plugins like mudflap create artificial variables/types
>> and expect type trees (*_type_node) have been created.
>> ?Therefore PLUGIN_START_UNIT callbacks are invoked after all the
>> frontend initialization but before code compilation.
>
> I like the patch. Would you mind just adding a small test for it?
> Anything that uses one of the _type_node nodes should be enough.
>
>> ?The patch bootstraps on x86_64-unknown-linux-gnu target and has no
>> test suite regressions.
>> ?Thank you
>>
>> tunji
>>
>
>
> Thanks,
> --
> Rafael Avila de Espindola
>
> Google | Gordon House | Barrow Street | Dublin 4 | Ireland
> Registered in Dublin, Ireland | Registration Number: 368047
>

Attachment: start_unit_plugin.txt
Description: Text document