[cfe-dev] Checker Plugins

Jordy Rose jediknil at belkadan.com
Thu Aug 4 20:29:51 PDT 2011


Whoops, forgot to include the removal of CheckerProvider from CheckerManager.cpp.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: CheckerPlugins.4.patch
Type: application/octet-stream
Size: 36593 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20110804/0ab5451e/attachment.obj>
-------------- next part --------------



On Aug 4, 2011, at 19:01, Jordy Rose wrote:

> Here's my latest work of CheckerPlugin, as per off-list discussion on what checker registration should look like. This time I haven't included any dynamic loading yet, only switched the current CheckerProvider interface over to a more useful CheckerRegistry. Plugin checker registration, based on a single per-library entry point (clang_getCheckers?), should pretty much not affect what's in this design at all.
> 
> The actual plugin loading code is blocked by the DynamicLibrary patch I sent to llvm-commits a few days ago.
> 
> There's a lot of infrastructure that might seem like premature optimization (in particular, keeping around the size of every package to avoid string comparisons when selecting checkers). This was mostly me freaking out about the speed hit on "make test" until I realized that my baseline was wrong. (I recently started building with CXX=clang++, Ted rewrote the CFG traversal, speed of a Debug+Asserts build isn't representative anyway...whatever.) So I can go ahead and take all that out again.
> 
> What do you think?
> Jordy
> 
> <CheckerPlugins.4.patch>



More information about the cfe-dev mailing list