[libc-commits] [PATCH] D77949: [libc] Add SIMD strlen implementation .

[Paula Toth via Phabricator via libc-commits]

PaulkaToast added a comment.

Hey tschuett, thank you for this contribution! (:

This is a great optimization and I believe it might be worth exploring if this change can be made in clang.
It would be fantastic if this could be recognized by the compiler and optimized there rather then having the logic here. 
Afaik one of the design goals of llvm-libc is to make the implementations as platform agnostic as possible, this means avoiding assembly and also letting the compiler handle optimizations from a platform to platform basis.

Just something to consider, I think it would be quite meaningful.


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

https://reviews.llvm.org/D77949