[PATCH] D51762: First part of the <chrono> calendar stuff
Eric Fiselier via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 11 10:24:39 PDT 2018
EricWF requested changes to this revision.
EricWF added a comment.
This revision now requires changes to proceed.
Have you run `git clang-format` over this change set?
The blocking issues I see are:
- The literal's need to be guarded against older clang dialects. So do their tests.
- There are a bunch of tests with meaningless `XFAIL` directives in them. They need to be removed.
================
Comment at: include/chrono:2667
+#if _LIBCPP_STD_VER > 17
+ constexpr chrono::day operator ""d(unsigned long long __d) noexcept
+ {
----------------
Including this file with Clang 6.0 in C++2a mode causes a compile error because of "-Wreserved-user-defined-literal". We need to wrap this block with:
```
#if defined(__clang__)
#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wreserved-user-defined-literal"
#endif
[...]
#if defined(__clang__)
#pragma clang diagnostic pop
#endif
```
https://reviews.llvm.org/D51762
More information about the cfe-commits
mailing list