[PATCH] D29898: [Support] Add format_provider for StringLiteral

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: include/llvm/Support/FormatProviders.h:49-50
     : public std::integral_constant<
-          bool, is_one_of<T, llvm::StringRef, std::string>::value ||
+          bool, is_one_of<T, llvm::StringRef, llvm::StringLiteral,
+                          std::string>::value ||
                     is_cstring<T>::value> {};
----------------
Perhaps this should use "is_convertible" which would match the implementation (well, not quite - the implementation should probably use "StringRef S = V" rather than StringRef S(V))?


https://reviews.llvm.org/D29898