[PATCH] D73027: [DependenceAnalysis] Dependecies for loads marked with "ivnariant.load" should not be shared with general accesses(PR42151).

[Johannes Doerfert via Phabricator via llvm-commits]

jdoerfert added a comment.

Just to make sure, this contains a test for the regression that was reported and lead to the initial revert, correct?



================
Comment at: llvm/lib/Analysis/MemoryDependenceAnalysis.cpp:1395
-    (void)foundBlock; (void)GotWorklistLimit;
-    assert((foundBlock || GotWorklistLimit) && "Current block not in cache?");
   }
----------------
Why is the `foundBlock` stuff gone?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73027