[PATCH] D26830: [libcxx] Add string_view literals

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 29 21:09:31 PST 2016


EricWF added a comment.

Alright. I committed the Clang changes. Just re-building now so I can test this.



================
Comment at: include/string_view:780
+} // namespace literals
+#endif
+
----------------
`// _LIBCPP_STD_VER > 14` comment please.


================
Comment at: test/std/strings/string.view/string.view.literals/literal1.pass.cpp:16
+
+int main()
+{
----------------
What's the point of this test that `literal.pass.cpp` doesn't cover?


================
Comment at: test/std/strings/string.view/string.view.literals/literal2.pass.cpp:16
+
+int main()
+{
----------------
What's the point of this test that `literal.pass.cpp` doesn't cover?


https://reviews.llvm.org/D26830





More information about the cfe-commits mailing list