C++ refactor: time getters now return milliseconds Previously they returned nanoseconds but all call sites converted those to milliseconds. This is not only a simplification of the semantics and renaming of the methods to make the returned units and the purpose clear, but also preparation for the Windows/MSVC implementations of these methods. -- MOS_MIGRATED_REVID=138383956
diff --git a/src/main/cpp/blaze.cc b/src/main/cpp/blaze.cc index 26053f6..da7b100 100644 --- a/src/main/cpp/blaze.cc +++ b/src/main/cpp/blaze.cc
@@ -655,7 +655,7 @@ } // Wall clock time since process startup. - globals->startup_time = ProcessClock() / 1000000LL; + globals->startup_time = GetMillisecondsSinceProcessStart(); if (VerboseLogging()) { fprintf(stderr, "Starting %s in batch mode.\n", @@ -985,15 +985,15 @@ // If the install dir doesn't exist, create it, if it does, we know it's good. struct stat buf; if (stat(globals->options->install_base.c_str(), &buf) == -1) { - uint64_t st = MonotonicClock(); + uint64_t st = GetMillisecondsMonotonic(); // Work in a temp dir to avoid races. string tmp_install = globals->options->install_base + ".tmp." + ToString(getpid()); string tmp_binaries = tmp_install + "/_embedded_binaries"; ActuallyExtractData(self_path, tmp_binaries); - uint64_t et = MonotonicClock(); - globals->extract_data_time = (et - st) / 1000000LL; + uint64_t et = GetMillisecondsMonotonic(); + globals->extract_data_time = et - st; // Now rename the completed installation to its final name. If this // fails due to an ENOTEMPTY then we assume another good @@ -1239,7 +1239,7 @@ } // Wall clock time since process startup. - globals->startup_time = ProcessClock() / 1000000LL; + globals->startup_time = GetMillisecondsSinceProcessStart(); // Unblock all signals. sigset_t sigset;