[PATCH] D35633: [ThinLTO] Add FunctionAttr NoRecurse and ReadAttr propagation to ThinLTO

[Charles Saternos via Phabricator via llvm-commits]

ncharlie added inline comments.


================
Comment at: lib/Transforms/IPO/FunctionImport.cpp:551
 
+void llvm::thinLTOInsertFunctionAttrsForModule(
+    Module &TheModule, const ModuleSummaryIndex &Index) {
----------------
Not sure if this is the best place for this function.


================
Comment at: lib/Transforms/IPO/ThinLTOBitcodeWriter.cpp:409
 
+static void computeThinMemoryAccess(Module &TheModule,
+                                    const ModuleSummaryIndex &Index,
----------------
Not sure if this is the best place for this function.


https://reviews.llvm.org/D35633