[PATCH] D36031: [sanitizer_common] Fuchsia OS support code
Vitaly Buka via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 28 16:24:53 PDT 2017
vitalybuka accepted this revision.
vitalybuka added a comment.
This revision is now accepted and ready to land.
I looked sanitizer_fuchsia.cc very briefly.
The rest LGTM
================
Comment at: lib/sanitizer_common/sanitizer_printf.cc:260
+ // Fuchsia's logging infrastructure always keeps track of the logging
+ // process, thread, and timestamp, so never prepend such information.
+ if (!SANITIZER_FUCHSIA && append_pid) {
----------------
I guess you would like to enabled as many as possible compiler-rt tests in future
So maybe better to avoid diverging if this is not critical?
Repository:
rL LLVM
https://reviews.llvm.org/D36031
More information about the llvm-commits
mailing list