[PATCH] [libcxx] Partially address a FIXME in steady_clock::now()

Dan Albert danalbert at google.com
Fri Aug 29 09:43:21 PDT 2014


================
Comment at: include/chrono:943
@@ -941,1 +942,3 @@
 typedef steady_clock high_resolution_clock;
+#else
+typedef system_clock high_resolution_clock;
----------------
Should there be a warning that the system won't use steady_clock?

================
Comment at: libcxx/test/utilities/time/time.clock/time.clock.steady/consistency.pass.cpp:14
@@ -13,2 +13,3 @@
 // XFAIL: with_system_lib=x86_64-apple-darwin12
+// XFAIL: no-monotonic-clock
 
----------------
I'd use UNSUPPORTED instead.

http://reviews.llvm.org/D4045






More information about the cfe-commits mailing list