Improve comments related to save_timestamp

(cherry picked from commit 0d0def13a52d7b68376636776e71a9af5bf01bb4)

diff --git a/src/ccache.cpp b/src/ccache.cpp
index 3cda1e4d6ac84dbe2f042a5f744ecbface2838f1..96c3972d3e45f9c21d26930f1d861a8644872c8c 100644 (file)
--- a/src/ccache.cpp
+++ b/src/ccache.cpp
@@ -1218,6 +1218,8 @@ update_manifest_file(void)
 
   auto old_st = Stat::stat(manifest_path);
 
+  // See comment in get_file_hash_index for why saving of timestamps is forced
+  // for precompiled headers.
   bool save_timestamp = (g_config.sloppiness() & SLOPPY_FILE_STAT_MATCHES)
                         || output_is_precompiled_header;
 

diff --git a/src/manifest.cpp b/src/manifest.cpp
index e634d8615eb81ef6128221edb0711f9257cfca1e..cbfe4f4791e40a8024768e17bf458236de71413d 100644 (file)
--- a/src/manifest.cpp
+++ b/src/manifest.cpp
@@ -223,10 +223,9 @@ private:
     // file_stat.ctime() may be 0, so we have to check time_of_compilation
     // against MAX(mtime, ctime).
     //
-    // ccache only reads mtime/ctime if sloppy_file_stat_match is setted,
-    // so mtimes/ctimes could store as a dummy value (-1) in other scenarios.
-    // This will effectively control the total number of file infos in
-    // certain scenarios, such as CI.
+    // ccache only reads mtime/ctime if file_stat_match sloppiness is enabled,
+    // so mtimes/ctimes are stored as a dummy value (-1) if not enabled. This
+    // reduces the number of file_info entries for the common case.
 
     auto file_stat = Stat::stat(path, Stat::OnError::log);
     if (file_stat) {