[PATCH] D148367: [ADT] add StringViewExtras llvm::starts_with for std::string_view

[Nick Desaulniers via Phabricator via llvm-commits]

nickdesaulniers added inline comments.


================
Comment at: llvm/include/llvm/ADT/StringViewExtras.h:24
+// is upgraded to C++20, remove this header and users.
+#ifdef __cpp_lib_starts_ends_with
+#warning "remove llvm::starts_with"
----------------
erichkeane wrote:
> This might be a bit noisy depending on how compilers/libraries define this in C++17 mode.  We might want a __cplusplus > (whatever the 2020 value is) here as well.
Yeah, I'll just remove it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148367