[PATCH] D115978: [RISCV] Use positive 0.0 for the neutral element in fadd reductions if nsz is present.

Roger Ferrer Ibanez via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 22 23:32:44 PST 2021


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

LGTM.

Though perhaps we can improve the generation of `-0.0` in another patch.

Something like this (similarly for `f64`)

  fmv.w.x	fa5, x0
  fsgnjn.s fa5, fa5, fa5

I think it should work and we avoid the constant pool (at expense of an extra instruction).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115978



More information about the llvm-commits mailing list