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

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 8 10:49:49 PDT 2021


phosek added inline comments.


================
Comment at: llvm/lib/Support/Caching.cpp:32
+                                             AddBufferFn AddBuffer,
+                                             StringRef CacheNameRef) {
   if (std::error_code EC = sys::fs::create_directories(CacheDirectoryPath))
----------------
Can we make this a `Twine`? I'd also make this argument the first one.


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