[PATCH] D131313: [Support] Use constexpr if

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 6 07:11:53 PDT 2022


nikic added a comment.

I don't think we should use `if constexpr` if we don't actually require it's semantics. A normal `if` will get optimized just fine. The only actual change here is that the llvm_is_multithreaded() definition gets moved into the header, which allows this to optimize without LTO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131313



More information about the llvm-commits mailing list