[PATCH] D95234: [RISCV] Define different pseudo instructions for different FPR.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 25 14:20:02 PST 2021


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVSDPatterns.td:42
 
+// We only model FPR32 for V instructions in RISCVInstrInfoV.td.
+// FP16/FP32/FP64 registers are alias each other. Convert FPR16 and FPR64
----------------
frasercrmck wrote:
> craig.topper wrote:
> > Please mark this with a FIXME to cleanup. Fraser or I can take care of it as a follow up.
> Yeah, no problem, I can take a look.
@frasercrmck I'm looking at this and might be proposing changes to this patch as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95234



More information about the llvm-commits mailing list