[PATCH] D41497: [ThinLTO][CachePruning] explicitly disable pruning

Pavel Labath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 21 09:34:31 PST 2017


labath added a comment.

This looks much cleaner indeed. lgtm, but please wait for signoff from the LTO folks as well.



================
Comment at: include/llvm/Support/CachePruning.h:31
   /// prune. A value of 0 forces the scan to occur.
-  std::chrono::seconds Interval = std::chrono::seconds(1200);
+  llvm::Optional<std::chrono::seconds> Interval = std::chrono::seconds(1200);
 
----------------
Maybe document the meaning of `llvm::None` ?


https://reviews.llvm.org/D41497





More information about the llvm-commits mailing list