[all-commits] [llvm/llvm-project] fdbff8: [RISCV][GISel] Add support for G_FCMP with F and D...

Craig Topper via All-commits all-commits at lists.llvm.org
Thu Nov 9 20:45:48 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fdbff88196b54cea12deb1fae978277066dffd1e
      https://github.com/llvm/llvm-project/commit/fdbff88196b54cea12deb1fae978277066dffd1e
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-11-09 (Thu, 09 Nov 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/GISel/RISCVInstructionSelector.cpp
    M llvm/lib/Target/RISCV/GISel/RISCVLegalizerInfo.cpp
    M llvm/lib/Target/RISCV/GISel/RISCVRegisterBankInfo.cpp
    A llvm/test/CodeGen/RISCV/GlobalISel/instruction-select/fcmp-rv32.mir
    A llvm/test/CodeGen/RISCV/GlobalISel/instruction-select/fcmp-rv64.mir
    A llvm/test/CodeGen/RISCV/GlobalISel/legalizer/rv32/legalize-fcmp.mir
    A llvm/test/CodeGen/RISCV/GlobalISel/legalizer/rv64/legalize-fcmp.mir
    A llvm/test/CodeGen/RISCV/GlobalISel/regbankselect/fcmp-rv32.mir
    A llvm/test/CodeGen/RISCV/GlobalISel/regbankselect/fcmp-rv64.mir

  Log Message:
  -----------
  [RISCV][GISel] Add support for G_FCMP with F and D extensions. (#70624)

We only have instructions for OEQ, OLT, and OLE. We need to convert
other comparison codes into those.

I think we'll likely want to split this up in the future to support
optimizations. Maybe do some of it in the legalizer or in a new post
legalizer lowering pass. So this patch is just enough to get something
working without adding 11 additional patterns to tablegen for each type.




More information about the All-commits mailing list