minor tweaks to constexpr_strlen() definition
minor tweaks to constexpr_strlen() definition

Summary:
- Use `FOLLLY_HAS_FEATURE(cxx_constexpr_string_builtins)` to check for the
  availability of `__builtin_strlen()` on clang, as recommended by the clang
  documentation (https://clang.llvm.org/docs/LanguageExtensions.html)
- Only use `std::strlen()` on gcc, rather than having checks to exclude all
  other compilers.  `strlen()` happens to be constexpr on gcc, but this is not
  required by the standard and cannot be assumed to be true in general.

Reviewed By: yfeldblum

Differential Revision: D6881815

fbshipit-source-id: 88c614419ae390464893087513dc764b0bfddfb8

facebook/folly,rklabs/folly,facebook/folly,rklabs/folly,rklabs/folly,facebook/folly,rklabs/folly,rklabs/folly,facebook/folly,facebook/folly