This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Synchronize clock selection between chrono.cpp and filesystem_clock.cpp
ClosedPublic

Authored by ldionne on Jul 4 2023, 11:55 AM.

Details

Summary

Note that _FilesystemClock will now be implemented by calling gettimeofday()
on Apple platforms instead of clock_gettime(). However, since both are
equivalent, this should not change the behavior on Apple platforms.
There should be no behavior change on other platforms.

In addition to being a consistency clean up, this fixes some issues seen
by folks as reported in https://reviews.llvm.org/D154390#4471924.

Diff Detail

Event Timeline

ldionne created this revision.Jul 4 2023, 11:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 4 2023, 11:55 AM
ldionne requested review of this revision.Jul 4 2023, 11:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 4 2023, 11:55 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
michaelplatings accepted this revision.Jul 4 2023, 12:16 PM
michaelplatings added a subscriber: michaelplatings.

The fixes building with picolibc, thanks!

ldionne accepted this revision as: Restricted Project.Jul 5 2023, 5:37 AM
This revision is now accepted and ready to land.Jul 5 2023, 5:37 AM