[patch] Rename CLANG_IS_PRODUCTION to CLANG_NO_PLUGIN_SUPPORT

NAKAMURA Takumi geek4civic at gmail.com
Sun Mar 9 10:18:22 PDT 2014


I suggest LLVM_ENABLE_PLUGIN (or LLVM_ENABLE_PLUGINS).
I am planning to propose plugins feature on win32.

2014-03-10 2:05 GMT+09:00 Tobias Grosser <tobias at grosser.es>:
> On 03/09/2014 04:08 PM, Rafael EspĂ­ndola wrote:
>>
>> The clang build has a CLANG_IS_PRODUCTION option. Historically this
>> would disable features that were not ready yet (c++ for example). It
>> would also disable plugins, which is an expensive feature for some
>> setups.
>>
>> Currently the option is only used to disable plugins, so the attached
>> patch renames it to CLANG_NO_PLUGIN_SUPPORT.
>
>
> What about using CLANG_PLUGIN_SUPPORT? This would avoid double negations.
>
>
>> On the Makefile build it also enables dead stripping in plugins are
>> disabled. It looks like it was an oversight when dead stripping
>> support was added.
>
>
> And just out of interest. Do you have an idea how much we loose in terms of
> start up performance, due to plugin support?
>
> Tobias
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits




More information about the cfe-commits mailing list