[PATCH] D96028: [RISCV] Add patterns for scalable-vector fabs & fcopysign

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2759
+      break;
+    // There is a form of VFSGNJ which injects the negated sign of its second
+    // operand. Try and bubble any FNEG up after the extend/round to produce
----------------
Should we avoid doing this if the extend/round is used elsewhere so we don't end up with extends/rounds of both the original and negated value?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96028