[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