[all-commits] [llvm/llvm-project] bf94ba: [RISCV][MC] Fix all remaining fcvt instructions th...

Alex Bradbury via All-commits all-commits at lists.llvm.org
Sun Oct 1 02:00:09 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: bf94ba39b65d1212ea84d5783b393280e1ce7478
      https://github.com/llvm/llvm-project/commit/bf94ba39b65d1212ea84d5783b393280e1ce7478
  Author: Alex Bradbury <asb at igalia.com>
  Date:   2023-10-01 (Sun, 01 Oct 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVInstrInfoD.td
    M llvm/lib/Target/RISCV/RISCVInstrInfoZfh.td
    M llvm/test/MC/RISCV/fp-default-rounding-mode.s
    M llvm/test/MC/RISCV/fp-inx-default-rounding-mode.s
    M llvm/test/MC/RISCV/rv32d-valid.s
    M llvm/test/MC/RISCV/rv32zdinx-valid.s
    M llvm/test/MC/RISCV/rv32zfhmin-valid.s
    M llvm/test/MC/RISCV/rv32zhinxmin-valid.s
    M llvm/test/MC/RISCV/rv64zhinxmin-valid.s

  Log Message:
  -----------
  [RISCV][MC] Fix all remaining fcvt instructions that didn't accept rounding mode but should have (#67889)

This is a follow-up to #67555, performing the same fix for the other
instructions that had this issue:
* fcvt.d.w
* fcvt.d.wu
* fcvt.s.h
* fcvt.d.h

As before, we stick to the 'rne' default because this gives maximum
compatibility with older LLVM and GNU tools when disassembling. I've
also double checked disassembling fp-default-rounding-mode.s with GNU
objdump to re-confirm it uses frm=0b000 for these instructions.




More information about the All-commits mailing list