bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.

Looks good with one comment.


================
Comment at: lib/StaticAnalyzer/Core/MemRegion.cpp:141-143
@@ -140,5 +141,1 @@
-MemRegionManager::~MemRegionManager() {
-  // All regions and their data are BumpPtrAllocated.  No need to call
-  // their destructors.
-}
 
----------------
Please keep the comments.


http://reviews.llvm.org/D13890



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

Reply via email to