[all-commits] [llvm/llvm-project] 39707c: [RISCV] Add test coverage for (select (icmp X, Y), ...

Craig Topper via All-commits all-commits at lists.llvm.org
Tue Aug 16 21:33:55 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 39707c1a9a9d88566a7d7939b5602c9d454276ef
      https://github.com/llvm/llvm-project/commit/39707c1a9a9d88566a7d7939b5602c9d454276ef
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-08-16 (Tue, 16 Aug 2022)

  Changed paths:
    A llvm/test/CodeGen/RISCV/double-select-icmp.ll
    A llvm/test/CodeGen/RISCV/float-select-icmp.ll
    A llvm/test/CodeGen/RISCV/half-select-icmp.ll

  Log Message:
  -----------
  [RISCV] Add test coverage for (select (icmp X, Y), float, float). NFC

We fold integer setcc into SELECT_CC during DAG combine even if
the SELECT_CC has FP result type, but we had no test coverage.


  Commit: d27c147aaa8fea0d3b438ba9e51b497a621ecf1e
      https://github.com/llvm/llvm-project/commit/d27c147aaa8fea0d3b438ba9e51b497a621ecf1e
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-08-16 (Tue, 16 Aug 2022)

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

  Log Message:
  -----------
  [RISCV] Allow lowerSELECT to fold integer setcc with FP select.

We'd pick it up in DAG combine later even if we didn't handle it here.
No test changes because we get it in DAG combine anyway.


Compare: https://github.com/llvm/llvm-project/compare/9fc65658f5e5...d27c147aaa8f


More information about the All-commits mailing list