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

[ben via Phabricator via llvm-commits]

bd1976llvm marked an inline comment as done.
bd1976llvm added inline comments.


================
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.
----------------
tejohnson wrote:
> bd1976llvm wrote:
> > tejohnson wrote:
> > > Wouldn't this need to be *Policy.Interval, like in the below <= check?
> > The != operator is overloaded so it does the right thing.
> Ah ok, see that. It looks like <= has the same overloads - can the * be removed there?
Nope. On some platforms, e.g. windows, the type of TimeStampModTime is nanoseconds. Therefore, none of the Optional <= operator overloads will match.


https://reviews.llvm.org/D41497