[libcxx-commits] [PATCH] D122397: [libc++] Use __builtin_expect and __builtin_assume in _LIBCPP_ASSERT

Arthur Eubanks via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Tue Jun 27 16:20:13 PDT 2023


aeubanks added a comment.

were there any benchmarks showing that this was beneficial? as discussed in https://discourse.llvm.org/t/llvm-assume-blocks-optimization/71609, sprinkling assumes everywhere is not necessarily a worthwhile tradeoff and perhaps the default should be to not add assumes everywhere


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122397



More information about the libcxx-commits mailing list