[PATCH] D51762: First part of the <chrono> calendar stuff

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 12 22:50:28 PDT 2018


EricWF added inline comments.


================
Comment at: test/std/utilities/time/time.cal/time.cal.ymwd/time.cal.ymwd.members/ctor.sys_days.pass.cpp:10
+// UNSUPPORTED: c++03, c++11, c++14, c++17
+// XFAIL
+
----------------
mclow.lists wrote:
> EricWF wrote:
> > What?
> The `sys_days` type is not implemented yet.
> That will be in the next patch.
> 
Right. But this `XFAIL` does nothing. The test suite only accepts `XFAIL: <condition>` for syntax.

If the `XFAIL` was actually considered, this test would fail as "unexpectedly passing"


https://reviews.llvm.org/D51762





More information about the cfe-commits mailing list