[PATCH] D87061: Handle masked loads and stores in MemoryLocation/Dependence

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 16:25:22 PDT 2020


efriedma added a comment.

Testcase?  It should be possible to show that GVN is more precise in cases involving masked load/store.



================
Comment at: llvm/lib/Analysis/MemoryDependenceAnalysis.cpp:472
+          continue;
+        return MemDepResult::getClobber(II);
+      }
----------------
What benefit are you getting here that's better than just falling through to the call to getModRefInfo()?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87061



More information about the llvm-commits mailing list