[PATCH] D112548: [LoopVectorize] Propagate fast-math flags for inloop reductions

Kerry McLaughlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 28 09:01:26 PDT 2021


kmclaughlin accepted this revision.
kmclaughlin added a comment.
This revision is now accepted and ready to land.

Thank you @RosieSumpter, this patch looks good to me and I think all of the comments have been addressed.



================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/strict-fadd.ll:854
+; CHECK-UNORDERED: [[FADD4]] = fadd nnan float [[LOAD4]], [[VEC_PHI4]]
+; CHECK-UNORDERED-NOT: call fast float @llvm.vector.reduce.fadd
+; CHECK-UNORDERED: middle.block:
----------------
Should `fast` here be replaced with `nnan`?


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

https://reviews.llvm.org/D112548



More information about the llvm-commits mailing list