[PATCH] D145900: [RISCV] Support ISD::STRICT_FADD/FSUB/FMUL/FDIV for vector types.

Yeting Kuo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 13 21:11:43 PDT 2023


fakepaper56 marked an inline comment as not done.
fakepaper56 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td:838
 defm : VPatWidenBinaryFPSDNode_VV_VF_WV_WF<fadd, "PseudoVFWADD">;
+defm : VPatWidenBinaryFPSDNode_VV_VF_WV_WF<fadd, "PseudoVFWADD">;
 defm : VPatWidenBinaryFPSDNode_VV_VF_WV_WF<fsub, "PseudoVFWSUB">;
----------------
fakepaper56 wrote:
> craig.topper wrote:
> > craig.topper wrote:
> > > This looks identical to the line above.
> > This line still looks identical to the line before it.
> Sorry, I mis-filled the done status.
The reason I didn't deal with widen operations now is we need to deal with `(any_fadd (any_fpextend))` and the test cases change may be too large to read. Is my concern over-thinking? Actually I don't know how large should a patch be ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145900



More information about the llvm-commits mailing list