[PATCH] D35704: added reset feature to dfsan

Farah Hariri via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 1 12:59:58 PDT 2017


farahhariri marked 2 inline comments as done.
farahhariri added inline comments.


================
Comment at: lib/dfsan/dfsan.cc:175
+
+  // reset potentially polluted union table entries
+  for (uptr l = 0; l <= last_label; ++l) {
----------------
pcc wrote:
> Is this part necessary? I believe that the mmap call above will also reset the union table to zero.
You're right it's not. Thanks!


https://reviews.llvm.org/D35704





More information about the llvm-commits mailing list