[PATCH] D157631: [LoopVectorize] Fix incorrect order of invariant stores when there are multiple reductions.
David Sherwood via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 22 06:49:53 PDT 2023
david-arm added inline comments.
Herald added a subscriber: sunshaoce.
================
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
----------------
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.
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