[PATCH] D23599: [ThinLTO] Add caching to the new LTO API

Dean Michael Berris via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 22:29:33 PDT 2016


dberris added a subscriber: dberris.

================
Comment at: include/llvm/LTO/Caching.h:38
@@ +37,3 @@
+  CacheObjectOutput(StringRef CacheDirectoryPath, 
+                    std::function<void(std::unique_ptr<MemoryBuffer>)> AddBuffer)
+      : CacheDirectoryPath(CacheDirectoryPath), AddBuffer(AddBuffer) {}
----------------
Is it worth typedef'ing `std::function<void(std::unique_ptr<MemoryBuffer>)>` such that you don't repeat yourself here?


https://reviews.llvm.org/D23599





More information about the llvm-commits mailing list