[all-commits] [llvm/llvm-project] 89173d: [RISCV] Remove duplicate code. NFC

Craig Topper via All-commits all-commits at lists.llvm.org
Thu Jul 28 10:05:39 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 89173dee719100d92c08864316fbc85258a054af
      https://github.com/llvm/llvm-project/commit/89173dee719100d92c08864316fbc85258a054af
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-07-28 (Thu, 28 Jul 2022)

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

  Log Message:
  -----------
  [RISCV] Remove duplicate code. NFC

The same operations are part of `FloatingPointVecReduceOps` a little
bit earlier.


  Commit: 2750873dfe0b1921d371c20387918fbb8eb7ad93
      https://github.com/llvm/llvm-project/commit/2750873dfe0b1921d371c20387918fbb8eb7ad93
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-07-28 (Thu, 28 Jul 2022)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/test/CodeGen/RISCV/rvv/fixed-vectors-fp.ll
    M llvm/test/CodeGen/RISCV/rvv/fround-sdnode.ll

  Log Message:
  -----------
  [RISCV] Update lowerFROUND to use masked instructions.

This avoids a vmerge at the end and avoids spurious fflags updates.
This isn't used for constrained intrinsic so we technically don't have
to worry about fflags, but it doesn't cost much to support it.

To support I've extend our FCOPYSIGN_VL node to support a passthru
operand. Similar to what was done for VRGATHER*_VL nodes.

I plan to do a similar update for trunc, floor, and ceil.

Reviewed By: reames, frasercrmck

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


Compare: https://github.com/llvm/llvm-project/compare/1422a9689d79...2750873dfe0b


More information about the All-commits mailing list