[PATCH] D119674: [BOLT] Make BinaryFunction::eraseInvalidBBs() thread-safe

[Vladislav Khmelevsky via Phabricator via llvm-commits]

yota9 added inline comments.


================
Comment at: bolt/lib/Core/BinaryFunction.cpp:352
+      if (BB->size()) {
+        std::unique_lock<std::shared_timed_mutex> Lock(BC.CtxMutex);
+        Bytes += BB->estimateSize();
----------------
Maybe move lock under estimateSize()? 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119674/new/

https://reviews.llvm.org/D119674