This revision was automatically updated to reflect the committed changes.
Closed by commit rL286349: Remove TimeValue usage from Core/Module (authored by
labath).
Changed prior to commit:
https://reviews.llvm.org/D26275?vs=77209=77323#toc
Repository:
rL LLVM
https://reviews.llvm.org/D26275
labath updated this revision to Diff 77209.
labath added a comment.
It turns out this was the only usage of TimeValue::Dump. I've changed that into
a
standalone function so it can be called without a TimeValue. Once Zachary lands
the formatting change in llvm I'd like add a more generic time
clayborg added a comment.
It would be great if we can match the old format somehow. It is definitely
clearer. If we can't, don't worry about it too much as it isn't shown by
default. Hopefully no one is text scraping.
https://reviews.llvm.org/D26275
labath created this revision.
labath added a reviewer: clayborg.
labath added a subscriber: lldb-commits.
The only interesting part here is that TimePoint and TimeValue have different
natural string representations, which affects "target modules list" output. It
is now "2016-07-09
