[PATCH] D105438: [LV] Re-generate check lines of some fragile tests

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/invariant-store-vectorization-2.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt < %s -licm -loop-vectorize -force-vector-width=4 -dce -instcombine -licm -S | FileCheck %s
+
----------------
The test should ideally not depend on `licm`. Can you update it so it does not require either LICM run?

Also, it seems this contains similar comments to `llvm/test/Transforms/LoopVectorize/invariant-store-vectorization.ll`. what's the difference?


================
Comment at: llvm/test/Transforms/LoopVectorize/invariant-store-vectorization-2.ll:15
+
+; Instcombine'd version of above test. Now the store is no longer of invariant
+; value.
----------------
there's no test above?


================
Comment at: llvm/test/Transforms/LoopVectorize/invariant-store-vectorization.ll:287
+  %i10 = load i32, i32* %i
+  %i11 = mul nsw i32 %i9, %i10
+  %i12 = srem i32 %i11, 65536
----------------
Why rename those?


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

https://reviews.llvm.org/D105438