zaks.anna added inline comments.

================
Comment at: test/Analysis/dispatch-data-leak.m:50
+
+  malloc_buf = malloc(1024);
+  data = dispatch_data_create(buf, 1024, dispatch_get_main_queue(), ^{}); // 
expected-warning{{Potential leak of memory pointed to by 'malloc_buf'}}
----------------
I think we should remove testing of malloc -it does not add anything here and 
is a bit confusing since one might  think that malloc_"but is related to the 
dispatch test case.


https://reviews.llvm.org/D27409



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

Reply via email to