[PATCH] D16176: Introduce stats and stats_client libraries.

Ivan Krasin via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 16:48:54 PST 2016


krasin accepted this revision.
This revision is now accepted and ready to land.

================
Comment at: lib/sanitizer_common/sanitizer_flags.cc:51
@@ -50,1 +50,3 @@
+// %p = pid
+void SubstituteForFlagValue(const char *s, char *out, uptr out_size) {
   char *out_end = out + out_size;
----------------
pcc wrote:
> krasin1 wrote:
> > why? not that I insist, I just probably don't understand the constraints and interested to learn about that.
> If the C++ standard library is compiled with sanitizer instrumentation, we would have a circular dependency between sanitizer_common and the stdlib.
You're right. Thank you for the reminder.


http://reviews.llvm.org/D16176





More information about the llvm-commits mailing list