[ThinLTO/gold] Perform cache pruning when cache directory specified

Summary:
As pointed out in the review for D37993, for consistency with other
linkers, gold plugin should perform cache pruning whenever there is a
cache directory specified, which will use the default cache policy.

Reviewers: pcc

Subscribers: llvm-commits, inglorion

Differential Revision: https://reviews.llvm.org/D43389

llvm-svn: 325830
This commit is contained in:
Teresa Johnson 2018-02-22 20:57:05 +00:00
parent a2cc3c055c
commit fd6fcbc006
2 changed files with 5 additions and 3 deletions

View File

@ -8,7 +8,8 @@
; RUN: --plugin-opt=cache-dir=%t.cache \
; RUN: -o %t3.o %t2.o %t.o
; RUN: ls %t.cache | count 0
; We should just get the timestamp file
; RUN: ls %t.cache | count 1
; Verify that enabling caching is working with module with hash.
@ -22,7 +23,8 @@
; RUN: --plugin-opt=cache-dir=%t.cache \
; RUN: -o %t3.o %t2.o %t.o
; RUN: ls %t.cache | count 2
; Two cached objects, plus a timestamp file
; RUN: ls %t.cache | count 3
; Create two files that would be removed by cache pruning due to age.

View File

@ -1014,7 +1014,7 @@ static ld_plugin_status cleanup_hook(void) {
}
// Prune cache
if (!options::cache_policy.empty()) {
if (!options::cache_dir.empty()) {
CachePruningPolicy policy = check(parseCachePruningPolicy(options::cache_policy));
pruneCache(options::cache_dir, policy);
}