[cfe-commits] [patch] -add-plugin

[Rafael Ávila de Espíndola]

On 11-01-23 3:43 PM, Nico Weber wrote:
> Here is a proof-of-concept ctags plugin in ~200 non-comment lines made
> possible by this patch: http://codepad.org/CjvHUEkG
> With this -add-plugin flag, it is trivial to run this plugin on large
> code bases such as llvm/clang itself, while the normal build process
> takes place.
>
> (http://codepad.org/Gwr9bDWR can be used to convert the resulting
> sqlite database into a tags file in the exuberant ctags format, which
> can be read by e.g. vim)
>
> While writing this demo, I realized that -plugin-arg does not work for
> plugins added by -add-plugin. If you think this is a good change, I
> will fix -plugin-arg for add-plugins in a follow-up change.

Just some notes since I am not familiar enough with the code to fully 
review it.

Doing extra work during a compilation is similar to the way gcc plugins 
work. There might be an argument for really easy to use libraries 
instead of plugins, but if doing plugins at all, I agree with you that 
this is how it should be done.

> Nico
>

Cheers,
Rafael