[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:31:30 PDT 2016


dberris added inline comments.

================
Comment at: include/llvm/LTO/Caching.h:36
@@ +35,3 @@
+  
+  // Cre
+  CacheObjectOutput(StringRef CacheDirectoryPath, 
----------------
Incomplete documentation?

================
Comment at: include/llvm/LTO/Caching.h:67
@@ +66,3 @@
+ *    the occupied space goes below a threshold.
+ * @{
+ */
----------------
Unsure what this means in Doxygen, but should there be a closing `@}` somewhere else?


https://reviews.llvm.org/D23599





More information about the llvm-commits mailing list