[PATCH] D32195: Object: Remove ModuleSummaryIndexObjectFile class.

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 28 09:18:55 PDT 2017


tejohnson added inline comments.


================
Comment at: clang/lib/CodeGen/BackendUtil.cpp:63
 
+static llvm::cl::opt<bool> IgnoreEmptyThinLTOIndexFile(
+    "ignore-empty-index-file", llvm::cl::ZeroOrMore,
----------------
Not sure llvm options are encouraged in clang (I only see a couple of cases currently).

Can you instead just move llvm::getModuleSummaryIndexForFile into BitcodeReader and avoid a lot of these changes?


https://reviews.llvm.org/D32195





More information about the llvm-commits mailing list