[PATCH] D54538: [LV] Avoid vectorizing unsafe dependencies in uniform address

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 15 08:01:02 PST 2018


anna marked 2 inline comments as done.
anna added inline comments.


================
Comment at: test/Transforms/LoopVectorize/invariant-store-vectorization.ll:555
+; to the same address
+define void @unsafe_dep_uniform_load_store(i32 %arg, i32 %arg1, i64 %arg2, i16* %arg3, i32 %arg4, i64 %arg5) {
+; CHECK-LABEL: unsafe_dep_uniform_load_store
----------------
Ayal wrote:
> Good to mention this test is related to pr39653.
> 
> With -force-vector-width=4 the test can be simplified; it probably took the additional instructions to convince LV's cost-model to vectorize the original loop on its own, w/o -force-vector-width.
I'd tried simplifying the test with force-vector-width, but it still avoided vectorization stating "unsafe memory operations". 


Repository:
  rL LLVM

https://reviews.llvm.org/D54538





More information about the llvm-commits mailing list