[PATCH] D146974: [test] A test case for D146958

Paul Osmialowski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 17 07:18:34 PDT 2023


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


================
Comment at: llvm/test/Transforms/LoopVectorize/vector-no-scevcheck.ll:15
+  %mul0 = mul nsw i64 %i, %val2
+  %arrayidx0 = getelementptr inbounds double, ptr %0, i64 %mul0
+  %mul1 = mul nsw i64 %i, %val1
----------------
fhahn wrote:
> Could you add a new function that's basically the same as `@foo` but has `%arrayidx0` in `%loop2.header`?
> Could you add a new function that's basically the same as @foo but has %arrayidx0 in %loop2.header?

This will not expose the same behavior of the D146958 patch we care about in this test (namely, `vector.scevcheck` will remain).



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146974



More information about the llvm-commits mailing list