[all-commits] [llvm/llvm-project] 427278: [RISCV] Remove pseudos for vwcvt.f.x(u) with round...

Craig Topper via All-commits all-commits at lists.llvm.org
Fri Jul 7 08:38:45 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 427278d11a1fd2c43d80cb37808f9ec90f474db7
      https://github.com/llvm/llvm-project/commit/427278d11a1fd2c43d80cb37808f9ec90f474db7
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-07-07 (Fri, 07 Jul 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td
    M llvm/lib/Target/RISCV/RISCVInstrInfoVVLPatterns.td

  Log Message:
  -----------
  [RISCV] Remove pseudos for vwcvt.f.x(u) with rounding mode.

vwcvt.f.x doesn't use rounding mode. The integer value fits in
the mantissa of a 2x larger FP type so no rounding is required.

I've remove the Uses = [FRM] that is also not needed.

I deleted the isel patterns. Alternatively, we could keep them and
drop the rounding mode immediate. The patterns are currently untested
so I chose to delete them. If they become needed in the future, we
can decide then if we should have the patterns or teach the node
creation to use the non-RM form for widening.

This reverts part of D142102.

Reviewed By: luke

Differential Revision: https://reviews.llvm.org/D154653




More information about the All-commits mailing list