[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