[PATCH] D111371: [Support][ThinLTO] Move ThinLTO caching to LLVM Support library.

Teresa Johnson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 8 10:57:29 PDT 2021


tejohnson added inline comments.


================
Comment at: llvm/include/llvm/Support/Caching.h:68
+/// already exist.
+Expected<NativeObjectCache> localCache(StringRef CacheDirectoryPath,
+                                       AddBufferFn AddBuffer,
----------------
Nit: document new parameter.


================
Comment at: llvm/lib/Support/Caching.cpp:36
 
+  SmallString<10> CacheName = CacheNameRef;
   return [=](unsigned Task, StringRef Key) -> AddStreamFn {
----------------
Is this copy necessary? I believe sys::path::append takes a Twine and eventually makes a copy of it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111371/new/

https://reviews.llvm.org/D111371



More information about the cfe-commits mailing list