<div dir="ltr">Sorry for the delay, I've seen the patch (and the following one, about supressions), but wasn't able to review it yet.</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 14, 2014 at 12:43 PM, Kuba Brecka <span dir="ltr"><<a href="mailto:kuba.brecka@gmail.com" target="_blank">kuba.brecka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This is a refactor patch that moves a couple of stacktrace taking and/or symbolizer-related functions from TSan, makes them generic and moves them into sanitizer_common. This is to prepare the functions for an upcoming patch implementing issue suppression mechanism into ASan. NFC intented.<br>
<br>
There is a couple of subtle changes, that I'd like to make sure we're fine with:<br>
<br>
* The `#ifdef TSAN_GO` changes into `#ifdef SANITIZER_GO` in the appropriate places.<br>
* The `internal_alloc(MBlockReportStack, sizeof(ReportStack));` becomes `InternalAlloc(sizeof(ReportStack));`, not sure if the `MBlockReportStack` is used for anything important.<br>
* `stack->pc` to `stack->info.address`, how could this event work before???<br>
* `bool strip_main` argument to `SymbolizeStack` whether it should try to strip the main function from the stack trace<br>
<br>
<a href="http://reviews.llvm.org/D6276" target="_blank">http://reviews.llvm.org/D6276</a><br>
<br>
Files:<br>
  lib/sanitizer_common/sanitizer_symbolizer.cc<br>
  lib/sanitizer_common/sanitizer_symbolizer.h<br>
  lib/tsan/rtl/tsan_report.cc<br>
  lib/tsan/rtl/tsan_report.h<br>
  lib/tsan/rtl/tsan_rtl_report.cc<br>
  lib/tsan/rtl/tsan_symbolize.cc<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</div>