[PATCH] D16115: [test-suite] Add ClangAnalyzerBenchmarks directory to test-suite repository
Matthias Braun via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 12 11:16:22 PST 2016
MatzeB added a comment.
Putting static-analyzer tests into the test-suite sounds good to me, the static analyzer is part of llvm so this should be fine. Though I'd like to wait a bit to hear other opinions on this matter.
As for the patch itself:
- I'd call the directory ClangAnalyzer
- For the external file you should provide a sha256 (or md5) sum and ideally check it in the download script, so other people can be sure that the reference results do indeed match the artifact being analyzed.
- Why is there a subdirectory "2016-01-11-134007-41609-1" in the RefScanBuildResults directory, that name appears arbitrary for reference results.
More information about the cfe-commits