[compiler-rt] [TSan][compiler-rt] Move symbolization in ReportRace outside of locks on Apple platforms (PR #149516)

Thurston Dang via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 28 11:08:55 PDT 2025


thurstond wrote:

>From the patch title, I was expecting this to be NFC-ish on non-Apple platforms, but IIUC this pull request contains two changes:
1) NFC-ish: for all platforms, defer symbolization (store stack ID etc. instead of symbolized equivalent; then call SymbolizeStackElems() as needed)
2) Apple-specific changes to release locks

Could you please split this into two separate patches? It's much easier to reason about the changes when they are in separate NFC-ish and `#if SANITIZER_APPLE` patches.


https://github.com/llvm/llvm-project/pull/149516


More information about the llvm-commits mailing list