[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

Thu Jul 18 05:24:56 PDT 2019

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

================
Comment at: test/Analysis/DependenceAnalysis/InvariantLoad.ll:1
+; RUN: opt < %s -passes=gvn -S | FileCheck %s
+
----------------
reames wrote:
> Stylistically, I'd suggest combining these two files into one and using update_test_checks.py to generate check lines.
> 
> And surely there is a more reduced test case possible?
Will work on  the first request.
Regarding tests reduction. It already took lot of efforts (not mine though :-)) to reduce tests to what we have today.  I don't believe they can be significantly reduced with preserving same semantics.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D64405