AntonBikineev added inline comments.
================
Comment at: include/string_view:749
+inline namespace literals
+{
----------------
EricWF wrote:
> If this is new to C++17 then the new declarations should be guarded by `#if
> _LIBCPP_VERSION > 14`.
Eric, I was thinking about it, but the fact that the whole string_view code is
not guarded by _LIBCPP_VERSION > 14 (I guess it's because it can be compiled
with lower Standards just fine) stopped me from doing that. I'm still not sure
though.
https://reviews.llvm.org/D26830
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
