[PATCH] D64405: [DependenceAnalysis] Dependecies for loads marked with "ivnariant.load" should not be shared with general accesses. Fix for https://bugs.llvm.org/show_bug.cgi?id=42151

Evgeniy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 9 21:10:59 PDT 2019


ebrevnov marked an inline comment as done.
ebrevnov added inline comments.


================
Comment at: lib/Analysis/MemoryDependenceAnalysis.cpp:987
+
+  // Due to a semantic of invariant load many dependencies are ignored. Thus
+  // results of dependence analysis can't be shared with general case.
----------------
jdoerfert wrote:
> "the semantics of invariant loads"
Thanks for catching this. Will fix.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64405





More information about the llvm-commits mailing list