[PATCH] D49504: Enable C++2a Chrono Literals

Marshall Clow via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 18 12:57:58 PDT 2018


mclow.lists added a comment.

This looks ok to me - I'm assuming that somewhere there's a test for `int foo = 0x123d`, so we're sure that didn't get broken.



================
Comment at: test/SemaCXX/cxx2a-user-defined-literals.cpp:8
+namespace std {
+  struct string {};
+  namespace chrono {
----------------
Do you need `string` here, or is this a copy-pasto?


Repository:
  rC Clang

https://reviews.llvm.org/D49504





More information about the cfe-commits mailing list