[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus updated this revision to Diff 197210. Szelethus added a comment. Uploaded the correct diff. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59465/new/ https://reviews.llvm.org/D59465 Files: test/Analysis/checker-plugins.c test/Analysis/plugins/CMakeLists.txt

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus planned changes to this revision. Szelethus added a comment. In D59465#1482302 , @whisperity wrote: > @Szelethus I know the dependent patch D59464 > will move `examples/analyzer-plugin` to >

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

whisperity added a comment. @Szelethus I know the dependent patch D59464 will move `examples/analyzer-plugin` to `test/Analysis/plugins/...`, but this patch still seems to affect `examples/`. Are you sure this is the right diff? Because you are adding brand

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus updated this revision to Diff 196080. Szelethus added a comment. Accidentally uploaded the wrong diff. Rebase on top of D59464 . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59465/new/ https://reviews.llvm.org/D59465 Files:

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus updated this revision to Diff 196079. Szelethus added a comment. - Move the existing plugin to `test/` - Implement the new plugin in there also CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59465/new/ https://reviews.llvm.org/D59465 Files: examples/CMakeLists.txt

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus planned changes to this revision. Szelethus added a comment. I'll move this, and the already existing plugin to test/ or unittest/. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59465/new/ https://reviews.llvm.org/D59465

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus added a comment. Would you be more comfortable with this patch I didn't touch the examples folder? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59465/new/ https://reviews.llvm.org/D59465 ___ cfe-commits

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

dcoughlin requested changes to this revision. dcoughlin added a comment. This revision now requires changes to proceed. I'm not sure we really want to add any more examples of plugins. Analyzer plugins aren't supported and likely never will be. We probably shouldn't give people false hope that

[PATCH] D59465: [analyzer] Add example plugin for checker option handling

Szelethus created this revision. Szelethus added reviewers: NoQ, xazax.hun, rnkovacs, baloghadamsoftware. Szelethus added a project: clang. Herald added subscribers: cfe-commits, Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, szepet, whisperity, mgorny. Repository: rC