[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-07 Thread Vitaly Buka via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG8f3e52538d77: [NFC][sanitizer] Remove global PersistentAllocator (authored by vitalybuka). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-07 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 377903. vitalybuka added a comment. fix rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111256/new/ https://reviews.llvm.org/D111256 Files: compiler-rt/lib/sanitizer_common/sanitizer_chained_origin_d

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-07 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 377900. vitalybuka marked an inline comment as done. vitalybuka added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111256/new/ https://reviews.llvm.org/D111256 Files: compiler-rt/lib/sa

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-07 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse accepted this revision. morehouse added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111256/new/ https://reviews.llvm.org/D111256 _

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-06 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka marked an inline comment as done. vitalybuka added inline comments. Comment at: compiler-rt/lib/sanitizer_common/sanitizer_persistent_allocator.h:61 uptr mem = (uptr)MmapOrDie(allocsz, "stack depot"); +atomic_fetch_add(&allocated_size, allocsz, memory_order_re

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-06 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 377714. vitalybuka added a comment. fixed size Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111256/new/ https://reviews.llvm.org/D111256 Files: clang/docs/tools/clang-formatted-files.txt compiler-rt/li

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-06 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 377711. vitalybuka added a comment. remove duplicate Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111256/new/ https://reviews.llvm.org/D111256 Files: clang/docs/tools/clang-formatted-files.txt compiler

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-06 Thread Matt Morehouse via Phabricator via cfe-commits
morehouse added inline comments. Comment at: compiler-rt/lib/sanitizer_common/sanitizer_persistent_allocator.h:61 uptr mem = (uptr)MmapOrDie(allocsz, "stack depot"); +atomic_fetch_add(&allocated_size, allocsz, memory_order_relaxed); atomic_store(®ion_end, mem + allo

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-06 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 377650. vitalybuka added a comment. update a test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111256/new/ https://reviews.llvm.org/D111256 Files: clang/docs/tools/clang-formatted-files.txt compiler-rt

[PATCH] D111256: [NFC][sanitizer] Remove global PersistentAllocator

2021-10-06 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka created this revision. vitalybuka added reviewers: morehouse, eugenis. Herald added a subscriber: mgorny. vitalybuka requested review of this revision. Herald added projects: clang, Sanitizers, LLVM. Herald added subscribers: llvm-commits, Sanitizers, cfe-commits. This way is easier to