[PATCH] D15871: [sanitizer coverage] coverage pc buffer

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 4 16:48:11 PST 2016


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

LGTM


================
Comment at: lib/sanitizer_common/sanitizer_coverage_libcdep.cc:218
@@ +217,3 @@
+        sizeof(uptr) * kPcArrayMaxSize, "CovInit::pc_buffer"));
+  else
+    pc_buffer = nullptr;
----------------
Style: I would prefer
X = nullptr;
if (cond())
  X = ... 


http://reviews.llvm.org/D15871





More information about the llvm-commits mailing list