[PATCH] D126772: [LoopVectorize] Add support for invariant stores of ordered reductions

Malhar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 07:13:26 PDT 2022


malharJ added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/strict-fadd.ll:1371
+; CHECK-ORDERED: middle.block
+; CHECK-ORDERED: store float %4, float* %[[DEST_PTR]]
+; CHECK-ORDERED: for.body
----------------
david-arm wrote:
> I think this should be something like
> 
>   store float %[[RDX]], float* %[[DEST_PTR]]
> 
> because otherwise the tests might break if the numbering changes in future.
good spot, somehow I seem to have missed it.
I've corrected this in the submitted patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126772



More information about the llvm-commits mailing list