[PATCH] D123473: [LICM] Only create load in pre-header when promoting load.

[Nuno Lopes via Phabricator via llvm-commits]

nlopes added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LICM.cpp:2165
+    // A new load is introduced, remove WriteOnly attribute from the function.
+    Preheader->getParent()->removeFnAttr(Attribute::WriteOnly);
+    PreheaderLoad =
----------------
we can't remove the writeonly attribute locally. We would need to remove it transitively as it's illegal for a writeonly function to call a non-writeonly function.
We need to bail out if a non-local store is needed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123473