[PATCH] D157631: [LoopVectorize] Fix incorrect order of invariant stores when there are multiple reductions.

Igor Kirillov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 29 07:25:46 PDT 2023


igor.kirillov marked an inline comment as done.
igor.kirillov added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll:582
   %sum.next = add nsw i32 %sum, %0
   store i32 %sum.next, ptr %dst, align 4
   %mul.next = mul nsw i32 %mul, %0
----------------
david-arm wrote:
> Is it worth also having a test where you're storing to different pointers, given that they could alias? We should still maintain the order in that case too.
Pre-committed two tests


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157631



More information about the llvm-commits mailing list