[PATCH] D83311: [CodeMoverUtils] Add optional data dependence checks using MSSA

Bardia Mahjour via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 20 21:16:21 PDT 2020


bmahjour added inline comments.


================
Comment at: llvm/lib/Transforms/Utils/CodeMoverUtils.cpp:255
+        // write after write dependency on same memory location
+        if (InstMA == CurInstLastDefMA)
+          return true;
----------------
We also need to check for `InstLastDefMA == CurInstLastDefMA` in case both definitions are clobbered by the same memory phi.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83311





More information about the llvm-commits mailing list