[PATCH] D35704: added reset feature to dfsan

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 8 10:53:46 PDT 2017


pcc added a comment.

Please add the other hooks in a separate patch.



================
Comment at: test/dfsan/reset_multithread.cc:19
+    dfsan_reset();
+    assert(dfsan_get_label_count() == 0);
+  }
----------------
Is this assert necessary? It's basically testing for the same thing as the assert in `dfsan_reset`, isn't it?


https://reviews.llvm.org/D35704





More information about the llvm-commits mailing list