[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