[PATCH] D76997: Fix StringRef::strLen in windows with clang++ C++17
    Reid Kleckner via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue May  5 16:12:47 PDT 2020
    
    
  
rnk added a comment.
If the compiler has the builtin, IMO we should prefer the builtin. It's always going to be better and faster than the library implementation. I would recommend reordering the ifdefs to prefer `__builtin_strlen` over the STL.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76997/new/
https://reviews.llvm.org/D76997
    
    
More information about the llvm-commits
mailing list