[PATCH] D110235: [LoopVectorize] Support reductions that store intermediary result
Igor Kirillov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 28 05:54:07 PDT 2022
igor.kirillov marked an inline comment as done.
igor.kirillov added inline comments.
================
Comment at: llvm/test/Transforms/LoopVectorize/reduction-with-invariant-store.ll:412
+;
+; for(int i=0; i < 1000; i++) {
+; sum += src[i];
----------------
fhahn wrote:
> Would it be possible to add runtime tests for the mis-compiles fixed to https://github.com/llvm/llvm-test-suite/tree/main/SingleSource/UnitTests/Vectorizer?
Added a simple test here - https://reviews.llvm.org/D124609
I don't feel like any other test adds more robustness, but if you have an idea what could be also added I'll gladly implement it!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D110235/new/
https://reviews.llvm.org/D110235
More information about the llvm-commits
mailing list