================ @@ -0,0 +1,18 @@ +add_distinct_clang_unittest(ClangScalableAnalysisFrameworkTests ---------------- nico wrote:
Why "Framework"? This tests clangAnalysisScalable, so the usual name would be AnalysisScalableTests, or with a Clang prefix if there's a naming conflict with another test (or I suppose for consistency with ClangAnalysisTests). I see that you have a ClangScalableAnalysisTests below that depends on this here, but why the two targets instead of just one? https://github.com/llvm/llvm-project/pull/169131 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
