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

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

Thanks this looks much nicer. Question below about the handling of the Optional though.

As an aside, it would be great to have similar handling in parseCachePruningPolicy(), which is used by gold-plugin and lld, and which gives an error when -1 is passed as the interval. But that doesn't need to be in this patch, just noting it here.



================
Comment at: lib/Support/CachePruning.cpp:170
+      return false;
     if (Policy.Interval != seconds(0)) {
       // Check whether the time stamp is older than our pruning interval.
----------------
Wouldn't this need to be *Policy.Interval, like in the below <= check?


https://reviews.llvm.org/D41497