[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

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 20 19:43:24 PDT 2019


jdoerfert added a comment.

I like the part about the cache: checking for all loads, caching for non-inv loads only. Though, I didn't even know what @reames mentioned before.
For this reason, and the fact that I don't understand the changes in line 1456ff, I think @reames should make sure this is good.


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