[libcxx-commits] [PATCH] D126882: [libc++] Do not yield from __sp_mut::lock()

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jun 3 10:56:02 PDT 2022


ldionne added inline comments.


================
Comment at: libcxx/src/memory.cpp:141
 
-static constexpr std::size_t __sp_mut_count = 16;
+static constexpr std::size_t __sp_mut_count = 32;
 static constinit __libcpp_mutex_t mut_back[__sp_mut_count] =
----------------
Mordante wrote:
> Since you change this value it might be a good moment to document why 32 is "the right" value.
Honestly, I just decided to double it to decrease contention a bit, but it wasn't done in any scientific kind of way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126882



More information about the libcxx-commits mailing list