[all-commits] [llvm/llvm-project] 2138ef: foo

Craig Topper via All-commits all-commits at lists.llvm.org
Mon Oct 3 12:13:37 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2138ef354a2a8be9f273804e1133a3b97a493f03
      https://github.com/llvm/llvm-project/commit/2138ef354a2a8be9f273804e1133a3b97a493f03
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-10-03 (Mon, 03 Oct 2022)

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

  Log Message:
  -----------
  foo


  Commit: 4c03c9f375f326a87065443d649c6568a4b7dd67
      https://github.com/llvm/llvm-project/commit/4c03c9f375f326a87065443d649c6568a4b7dd67
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2022-10-03 (Mon, 03 Oct 2022)

  Changed paths:
    M llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/test/CodeGen/RISCV/rvv/stepvector.ll

  Log Message:
  -----------
  [RISCV] Add missing VL arguments to the creation of RISCVISD::VMV_V_X_VL nodes.

VMV_V_X_VL nodes should always have a passthru, a splat, and a VL.
We were sometimes missing the VL.

This went unnoticed because these cases were all selected into the
following node to form a .vx or .vi instruction. The ComplexPattern
that does this, doesn't check the VL operand. I've added an assert
to the ComplexPattern to catch if the operand is missing.

@qcolombet spotted some of these in D134703.


Compare: https://github.com/llvm/llvm-project/compare/efa110815ae9...4c03c9f375f3


More information about the All-commits mailing list