Revision tags: llvmorg-18.1.8, llvmorg-18.1.7, llvmorg-18.1.6, llvmorg-18.1.5, llvmorg-18.1.4, llvmorg-18.1.3, llvmorg-18.1.2, llvmorg-18.1.1, llvmorg-18.1.0, llvmorg-18.1.0-rc4, llvmorg-18.1.0-rc3, llvmorg-18.1.0-rc2, llvmorg-18.1.0-rc1, llvmorg-19-init |
|
#
0e07bf91 |
| 24-Dec-2023 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Rename to Lock{Before,After}Fork StackDepotBase locking (#76303)
Followup to #76279
|
#
f78a742a |
| 23-Dec-2023 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Rename Lock{Before,After}Fork suffixes locking StackDepotBase (#76279)
This is preparation for performance optimization.
We need to highlight that this is very specific lock, and
[NFC][sanitizer] Rename Lock{Before,After}Fork suffixes locking StackDepotBase (#76279)
This is preparation for performance optimization.
We need to highlight that this is very specific lock, and should not be
used for other purposes.
Add `fork_child` parameter to distinguish processes after fork.
show more ...
|
Revision tags: llvmorg-17.0.6, llvmorg-17.0.5, llvmorg-17.0.4, llvmorg-17.0.3, llvmorg-17.0.2, llvmorg-17.0.1, llvmorg-17.0.0, llvmorg-17.0.0-rc4, llvmorg-17.0.0-rc3, llvmorg-17.0.0-rc2, llvmorg-17.0.0-rc1, llvmorg-18-init, llvmorg-16.0.6, llvmorg-16.0.5, llvmorg-16.0.4, llvmorg-16.0.3, llvmorg-16.0.2, llvmorg-16.0.1, llvmorg-16.0.0, llvmorg-16.0.0-rc4, llvmorg-16.0.0-rc3, llvmorg-16.0.0-rc2, llvmorg-16.0.0-rc1, llvmorg-17-init, llvmorg-15.0.7, llvmorg-15.0.6, llvmorg-15.0.5, llvmorg-15.0.4, llvmorg-15.0.3, working, llvmorg-15.0.2, llvmorg-15.0.1, llvmorg-15.0.0, llvmorg-15.0.0-rc3, llvmorg-15.0.0-rc2, llvmorg-15.0.0-rc1, llvmorg-16-init, llvmorg-14.0.6, llvmorg-14.0.5, llvmorg-14.0.4, llvmorg-14.0.3, llvmorg-14.0.2, llvmorg-14.0.1, llvmorg-14.0.0, llvmorg-14.0.0-rc4, llvmorg-14.0.0-rc3, llvmorg-14.0.0-rc2 |
|
#
f978497d |
| 25-Feb-2022 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Don't collect unused info
|
Revision tags: llvmorg-14.0.0-rc1, llvmorg-15-init, llvmorg-13.0.1, llvmorg-13.0.1-rc3, llvmorg-13.0.1-rc2 |
|
#
765921de |
| 05-Jan-2022 |
Dmitry Vyukov <dvyukov@google.com> |
sanitizer_common: prefix thread-safety macros with SANITIZER_
Currently we use very common names for macros like ACQUIRE/RELEASE, which cause conflicts with system headers. Prefix all macros with SA
sanitizer_common: prefix thread-safety macros with SANITIZER_
Currently we use very common names for macros like ACQUIRE/RELEASE, which cause conflicts with system headers. Prefix all macros with SANITIZER_ to avoid conflicts.
Reviewed By: vitalybuka
Differential Revision: https://reviews.llvm.org/D116652
show more ...
|
Revision tags: llvmorg-13.0.1-rc1 |
|
#
fa24c4a1 |
| 23-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Run Stack compression in background thread
Depends on D114495.
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm.org/D114498
|
#
a1e7f628 |
| 09-Dec-2021 |
Petr Hosek <phosek@google.com> |
Revert "[sanitizer] Run Stack compression in background thread"
This reverts commit e5c2a46c5e8fc038b9f6c898df9628f9524dc10e as this change introduced a linker error when building sanitizer runtimes
Revert "[sanitizer] Run Stack compression in background thread"
This reverts commit e5c2a46c5e8fc038b9f6c898df9628f9524dc10e as this change introduced a linker error when building sanitizer runtimes:
ld.lld: error: undefined symbol: __sanitizer::internal_start_thread(void* (*)(void*), void*) >>> referenced by sanitizer_stackdepot.cpp:133 (compiler-rt/lib/sanitizer_common/sanitizer_stackdepot.cpp:133) >>> compiler-rt/lib/sanitizer_common/CMakeFiles/RTSanitizerCommonSymbolizer.x86_64.dir/sanitizer_stackdepot.cpp.obj:(__sanitizer::(anonymous namespace)::CompressThread::NewWorkNotify())
show more ...
|
#
e5c2a46c |
| 23-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Run Stack compression in background thread
Depends on D114495.
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm.org/D114498
|
#
81f9dc8e |
| 07-Dec-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Lock/Unlock stack store on fork
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm.org/D115210
|
#
7151c714 |
| 07-Dec-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Fix CompressStackStore VPrint message
|
#
7185b1f4 |
| 24-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Add compress_stack_depot flag
Depends on D114494.
Reviewed By: morehouse
Differential Revision: https://reviews.llvm.org/D114495
|
#
f9d585d0 |
| 02-Dec-2021 |
Steven Wan <wanyu9511@gmail.com> |
Revert "[sanitizer] Add compress_stack_depot flag"
This is failing on clang-s390x-linux, https://lab.llvm.org/buildbot/#/builders/94/builds/6748. This reverts commit bf18253b0ee543f98119e5ab6a5b57d0
Revert "[sanitizer] Add compress_stack_depot flag"
This is failing on clang-s390x-linux, https://lab.llvm.org/buildbot/#/builders/94/builds/6748. This reverts commit bf18253b0ee543f98119e5ab6a5b57d05c24d314.
show more ...
|
#
bf18253b |
| 24-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Add compress_stack_depot flag
Depends on D114494.
Reviewed By: morehouse
Differential Revision: https://reviews.llvm.org/D114495
|
#
7c1d7779 |
| 24-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Add entry point for compression
Add Compression::Test type which just pretends packing, but does nothing useful. It's only called from test for now.
Depends on D114493.
Reviewed B
[NFC][sanitizer] Add entry point for compression
Add Compression::Test type which just pretends packing, but does nothing useful. It's only called from test for now.
Depends on D114493.
Reviewed By: kstoimenov
Differential Revision: https://reviews.llvm.org/D114494
show more ...
|
#
a06d3527 |
| 23-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Track progress of populating the block
In multi-threaded application concurrent StackStore::Store may finish in order different from assigned Id. So we can't assume that after we sw
[NFC][sanitizer] Track progress of populating the block
In multi-threaded application concurrent StackStore::Store may finish in order different from assigned Id. So we can't assume that after we switch writing the next block the previous is done.
The workaround is to count exact number of uptr stored into the block, including skipped tail/head which were not able to fit entire trace.
Depends on D114490.
Reviewed By: morehouse
Differential Revision: https://reviews.llvm.org/D114493
show more ...
|
#
168bc7ce |
| 23-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Remove storeIds and use padding of StackDepotNode
Depends on D114489.
Reviewed By: morehouse, dvyukov
Differential Revision: https://reviews.llvm.org/D114490
|
#
d591a46d |
| 19-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Fix naming in StackStore
|
#
bb50e971 |
| 15-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Change StackStore API to use StackTrace
|
#
368a3b52 |
| 15-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Rename PersistentAllocator to StackStore
|
#
930ccf01 |
| 15-Nov-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Rename persistent_allocator to stack_store
|
#
85021768 |
| 12-Oct-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Remove tag from StackDepotNode
And share storage with size.
Depends on D111615.
Differential Revision: https://reviews.llvm.org/D111616
|
#
e5859afa |
| 16-Oct-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Remove use_count from StackDepotNode
This is msan/dfsan data which does not need waste cache of other sanitizers.
Depends on D111614.
Differential Revision: https://reviews.llvm.org/D1
[sanitizer] Remove use_count from StackDepotNode
This is msan/dfsan data which does not need waste cache of other sanitizers.
Depends on D111614.
Differential Revision: https://reviews.llvm.org/D111615
show more ...
|
#
746dd6a7 |
| 16-Oct-2021 |
Vitaly Buka <vitalybuka@google.com> |
[NFC][sanitizer] Add StackDepotTestOnlyUnmap
|
#
8282024a |
| 12-Oct-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Move out stack trace pointer from header StackDepot
Trace pointers accessed very rarely and don't need to be in hot data.
Depends on D111613.
Reviewed By: dvyukov
Differential Revisio
[sanitizer] Move out stack trace pointer from header StackDepot
Trace pointers accessed very rarely and don't need to be in hot data.
Depends on D111613.
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm.org/D111614
show more ...
|
#
ca0036df |
| 12-Oct-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Remove StackDepotReverseMap
Now StackDepotGet can retrive the stack in O(1).
Depends on D111612.
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm.org/D111613
|
#
ce7f8c84 |
| 12-Oct-2021 |
Vitaly Buka <vitalybuka@google.com> |
[sanitizer] Remove id and replace link with u32
This lets us reduce size of Node, similar to D111183 proposal.
Depends on D111610.
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm
[sanitizer] Remove id and replace link with u32
This lets us reduce size of Node, similar to D111183 proposal.
Depends on D111610.
Reviewed By: dvyukov
Differential Revision: https://reviews.llvm.org/D111612
show more ...
|