[Lldb-commits] [lldb] Fix MSVC build issues (PR #84362)

[Saleem Abdulrasool via lldb-commits]

================
@@ -168,8 +168,8 @@ class ConstString {
   // Implicitly convert \class ConstString instances to \class StringRef.
   operator llvm::StringRef() const { return GetStringRef(); }
 
-  // Implicitly convert \class ConstString instances to \class std::string_view.
-  operator std::string_view() const {
+  // Explicitly convert \class ConstString instances to \class std::string_view.
+  explicit operator std::string_view() const {
----------------
compnerd wrote:

How about flipping this to make `llvm::StringRef` the explicit conversion and leave `std::string_view` the implicit one? This seems like it would be better as we tend towards the standard types, which would make the eventual removal of `StringRef` easier.

https://github.com/llvm/llvm-project/pull/84362