[PATCH] D84589: [CodeMoverUtils] Add optional data dependence checks using Alias Analysis

Bardia Mahjour via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 18 14:03:27 PDT 2020


bmahjour added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/CodeMoverUtils.cpp:250
+  if (CallInst *CI = dyn_cast<CallInst>(&I)) {
+    for (Value *Op : CI->arg_operands())
+      if (Op->getType()->isPointerTy()) {
----------------
I think the call itself should also go into the MemLocs vector.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84589



More information about the llvm-commits mailing list