[PATCH] D37993: [ThinLTO/gold] Implement ThinLTO cache pruning support
Peter Collingbourne via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 31 17:18:01 PDT 2017
pcc added a comment.
In https://reviews.llvm.org/D37993#893466, @pcc wrote:
> Is it a good idea to support cache pruning in the gold plugin? As I mentioned in https://reviews.llvm.org/D31063 there are some unavoidable race conditions due to limitations in the plugin API.
Ping... not sure whether it's a good idea to ship a known racy feature.
If you really want this feature in gold I'd suggest extending the gold plugin API so that it accepts file descriptors, and then have the plugin support this feature conditionally on whether the gold version is new enough.
Repository:
rL LLVM
https://reviews.llvm.org/D37993
More information about the llvm-commits
mailing list