[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