Szelethus added a comment. I only looked at the checker naming issue, and that seems to have been resolved perfectly, thanks! :)
================ Comment at: clang/include/clang/StaticAnalyzer/Checkers/Checkers.td:582 + "false", + InAlpha>, + ]>, ---------------- Let's `Hide` this in addition. ================ Comment at: clang/test/Analysis/smart-ptr.cpp:6 +// RUN: %clang_analyze_cc1 -analyzer-checker=core\ +// RUN: -analyzer-checker cplusplus.SmartPtrModeling,alpha.cplusplus.SmartPtr\ +// RUN: -analyzer-config cplusplus.SmartPtrModeling:ModelSmartPtrDereference=false\ ---------------- `alpha.cplusplus.SmartPtr` on its own should be sufficient, dependencies ensure that `cplusplus.SmartPtrModeling` will be enabled and registered beforehand. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81315/new/ https://reviews.llvm.org/D81315 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits