This revision was automatically updated to reflect the committed changes.
Closed by commit rC335740: [analyzer] Allow registering custom
statically-linked analyzer checkers (authored by alexfh, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D45718?vs=152317=153091#toc
george.karpenkov accepted this revision.
george.karpenkov added a comment.
This revision is now accepted and ready to land.
LGTM
Repository:
rC Clang
https://reviews.llvm.org/D45718
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
alexfh added a comment.
Please take another look.
Repository:
rC Clang
https://reviews.llvm.org/D45718
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh updated this revision to Diff 152317.
alexfh added a comment.
Herald added subscribers: mikhail.ramalho, mgorny.
- Added a test.
Repository:
rC Clang
https://reviews.llvm.org/D45718
Files:
include/clang/StaticAnalyzer/Frontend/AnalysisConsumer.h
alexfh added a comment.
In https://reviews.llvm.org/D45718#1074611, @george.karpenkov wrote:
> Actually, we do have unittests in `tools/clang/unittest/Analysis`.
> @alexfh would you be able to write a unittest in there? It should be fairly
> easy following the structure of e.g.
>
george.karpenkov requested changes to this revision.
george.karpenkov added a comment.
This revision now requires changes to proceed.
Actually, we do have unittests in `tools/clang/unittest/Analysis`.
@alexfh would you be able to write a unittest in there? It should be fairly
easy following the