[all-commits] [llvm/llvm-project] c09730: [RISCV] Pre-commit tests for D145897. NFC

Craig Topper via All-commits all-commits at lists.llvm.org
Tue Mar 14 13:11:26 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c09730c23e207accbf487a5142647c118630a5d2
      https://github.com/llvm/llvm-project/commit/c09730c23e207accbf487a5142647c118630a5d2
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-03-14 (Tue, 14 Mar 2023)

  Changed paths:
    M llvm/test/CodeGen/RISCV/double-zfa.ll
    M llvm/test/CodeGen/RISCV/half-zfa-fli.ll

  Log Message:
  -----------
  [RISCV] Pre-commit tests for D145897. NFC


  Commit: a1e39f35c50a5292f464cf59aa63b8ed0107dfce
      https://github.com/llvm/llvm-project/commit/a1e39f35c50a5292f464cf59aa63b8ed0107dfce
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-03-14 (Tue, 14 Mar 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
    M llvm/lib/Target/RISCV/MCTargetDesc/RISCVBaseInfo.cpp
    M llvm/lib/Target/RISCV/MCTargetDesc/RISCVBaseInfo.h
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/lib/Target/RISCV/RISCVInstrInfoZfa.td

  Log Message:
  -----------
  [RISCV] Merge getLoadFP*Imm into a single function.

We currently have 3 functions and 3 lookup tables. This was the
most expediant and obvious way to fix several bugs.

This patch uses a single function and single lookup
table. It uses APFloat::convert to convert from the half or double
to single precision. If the conversion doesn't have any errors or
lose any information we use the f32 table to finish the lookup.

Reviewed By: asb

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


Compare: https://github.com/llvm/llvm-project/compare/1c5d12144058...a1e39f35c50a


More information about the All-commits mailing list