[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