[PATCH] D70984: [ThinLTO] Add option to disable readonly/writeonly attribute propagation

[Steven Wu via Phabricator via llvm-commits]

steven_wu added inline comments.


================
Comment at: llvm/lib/IR/ModuleSummaryIndex.cpp:166
+    bool ImportEnabled) {
+  setWithAttributePropagation();
+
----------------
Do we still need to drop all the readonly writeonly flag if simply return before `setWithAttributePropagation()`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70984/new/

https://reviews.llvm.org/D70984