[PATCH] D38992: Support formatv of TimePoint with strftime-style formats.

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 17 06:07:18 PDT 2017


labath added a comment.

I wanted to add something like this myself, but never got around to doing that. Thanks.



================
Comment at: lib/Support/Chrono.cpp:65
+  for (unsigned I = 0; I < Style.size(); ++I) {
+    if (Style[I] == '%' && Style.size() > I + 1) switch (Style[I + 1]) {
+        case 'L':  // Milliseconds, from Ruby.
----------------
This won't correctly handle the case when the `%` is already escaped (e.g. `%%foobar`). 


https://reviews.llvm.org/D38992





More information about the llvm-commits mailing list