[PATCH] D29459: [asan] Move `DescribeSignalOrException` to sanitizer common.

Marcos Pividori via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 2 10:57:03 PST 2017


mpividori added inline comments.


================
Comment at: lib/asan/asan_errors.cc:64
   Printf("%s", d.Warning());
-  const char *description = DescribeSignalOrException(signo);
+  const char *description = __sanitizer::DescribeSignalOrException(signo);
   Report(
----------------
kubamracek wrote:
> Why do we need `__sanitizer::`?  Other parts of ASan call sanitizer_common code all the time without this.
You are right. Is not necessary.


https://reviews.llvm.org/D29459





More information about the llvm-commits mailing list