kcc added inline comments.

================
Comment at: clang/lib/Driver/SanitizerArgs.cpp:297
                             CoverageTraceCmp | CoveragePCTable;
+#if !defined(__APPLE__)
+        // Due to TLS differences, stack depth tracking is disabled on Mac.
----------------
please use if(SomeCondition) instead of #if

In general: 99% of cases where you may want to use #if -- you shouldn't


================
Comment at: 
compiler-rt/lib/sanitizer_common/sanitizer_coverage_libcdep_new.cc:20
 #include "sanitizer_symbolizer.h"
+#include <cstdint>
 
----------------
no standard hearder in this files. Just use the 'uptr' type. 


https://reviews.llvm.org/D37156



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to