vrnithinkumar marked 4 inline comments as done.
vrnithinkumar added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Checkers/Checkers.td:582
+                  "false",
+                  InAlpha>,
+  ]>,
----------------
Szelethus wrote:
> Let's `Hide` this in addition.
Added `Hide`


================
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\
----------------
Szelethus wrote:
> `alpha.cplusplus.SmartPtr` on its own should be sufficient, dependencies 
> ensure that `cplusplus.SmartPtrModeling` will be enabled and registered 
> beforehand.
Removed `cplusplus.SmartPtrModeling`


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

Reply via email to