xazax.hun added inline comments.

================
Comment at: lib/StaticAnalyzer/Checkers/MallocChecker.cpp:1003
@@ +1002,3 @@
+//
+ProgramStateRef MallocChecker::addExtentSize(CheckerContext &C,
+                                             const CXXNewExpr *NE,
----------------
zaks.anna wrote:
> I am not sure this code belongs to the malloc checker since it only supports 
> the array bounds checker. Is there a reason it's not part of that checker?
I think it is part of the malloc checker because it already does something very 
very similar to malloc, see the MallocMemAux function. So in fact, for the 
array bounds checker to work properly, the malloc checker should be turned on.


https://reviews.llvm.org/D24307



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to