[PATCH] D12771: [LICM] Hoist calls to readonly argmemonly functions even with stores in the loop

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 16 19:53:47 PDT 2015


majnemer added inline comments.

================
Comment at: lib/Transforms/Scalar/LICM.cpp:468-471
@@ +467,6 @@
+      if (AliasAnalysis::onlyAccessesArgPointees(Behavior)) {
+        for (Value *Op : CI->arg_operands())
+          if (Op->getType()->isPointerTy() &&
+              pointerInvalidatedByLoop(Op, MemoryLocation::UnknownSize,
+                                       AAMDNodes(), CurAST))
+            return false;
----------------
Is the function permitted to bitcast it's non-pointer parameters and store to them?


http://reviews.llvm.org/D12771





More information about the llvm-commits mailing list