[PATCH] [Sanitizer RT] Put the module name string ownership in Symbolizer in order

Timur Iskhodzhanov timurrrr at google.com
Mon Mar 30 08:21:33 PDT 2015


Like this?

Please note we have a warning if a function takes more than 512 bytes on stack, so I used `InternalScopedString` instead.  This implies `mmap`+`munmap` on each TU on process startup, plus a few times on shutdown.  I've tried the patch on `pdfium_test` and `chrome` and haven't observed much slowdown...

[TODO: update subject before comitting]


http://reviews.llvm.org/D8666

Files:
  lib/asan/asan_suppressions.cc
  lib/lsan/lsan_common.cc
  lib/sanitizer_common/sanitizer_coverage_libcdep.cc
  lib/sanitizer_common/sanitizer_symbolizer.cc
  lib/sanitizer_common/sanitizer_symbolizer.h
  lib/sanitizer_common/sanitizer_symbolizer_posix_libcdep.cc

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8666.22889.patch
Type: text/x-patch
Size: 8590 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150330/7528aa7a/attachment.bin>


More information about the llvm-commits mailing list