diff options
author | Dmitry Vyukov <dvyukov@google.com> | 2019-09-18 09:18:04 +0000 |
---|---|---|
committer | Dmitry Vyukov <dvyukov@google.com> | 2019-09-18 09:18:04 +0000 |
commit | 121faed8f831cc672d10460e0ae2e65296b3e0d4 (patch) | |
tree | 04babb5f58341a3c896fa25df0b4d0b905169d1c | |
parent | d5f68535b6a1fbb4f8afed3d999d965861bb8a4d (diff) | |
download | compiler-rt-121faed8f831cc672d10460e0ae2e65296b3e0d4.tar.gz |
tsan: allow the Go runtime to return multiple stack frames for a single PC
This fix allows tsan to report stack traces correctly even in the
presence of mid-stack inlining by the Go compiler.
See https://go-review.googlesource.com/c/go/+/195781 for the Go runtime side of this change.
Author: randall77 (Keith Randall)
Reviewed: https://reviews.llvm.org/D67671
git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@372205 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/tsan/go/tsan_go.cpp | 25 |
1 files changed, 18 insertions, 7 deletions
diff --git a/lib/tsan/go/tsan_go.cpp b/lib/tsan/go/tsan_go.cpp index 271f6bc00..f5998c0c7 100644 --- a/lib/tsan/go/tsan_go.cpp +++ b/lib/tsan/go/tsan_go.cpp @@ -54,20 +54,31 @@ struct SymbolizeCodeContext { }; SymbolizedStack *SymbolizeCode(uptr addr) { - SymbolizedStack *s = SymbolizedStack::New(addr); - SymbolizeCodeContext cbctx; - internal_memset(&cbctx, 0, sizeof(cbctx)); - cbctx.pc = addr; - go_runtime_cb(CallbackSymbolizeCode, &cbctx); - if (cbctx.res) { + SymbolizedStack *first = SymbolizedStack::New(addr); + SymbolizedStack *s = first; + for (;;) { + SymbolizeCodeContext cbctx; + internal_memset(&cbctx, 0, sizeof(cbctx)); + cbctx.pc = addr; + go_runtime_cb(CallbackSymbolizeCode, &cbctx); + if (cbctx.res == 0) + break; AddressInfo &info = s->info; info.module_offset = cbctx.off; info.function = internal_strdup(cbctx.func ? cbctx.func : "??"); info.file = internal_strdup(cbctx.file ? cbctx.file : "-"); info.line = cbctx.line; info.column = 0; + + if (cbctx.pc == addr) // outermost (non-inlined) function + break; + addr = cbctx.pc; + // Allocate a stack entry for the parent of the inlined function. + SymbolizedStack *s2 = SymbolizedStack::New(addr); + s->next = s2; + s = s2; } - return s; + return first; } struct SymbolizeDataContext { |