[all-commits] [llvm/llvm-project] c6b38f: [RISCV] Make sure we use LMUL=1 for scalar reducti...

Craig Topper via All-commits all-commits at lists.llvm.org
Wed Jan 11 16:25:52 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c6b38f491fb22eb5705bf30fed1ce4a14416e8fe
      https://github.com/llvm/llvm-project/commit/c6b38f491fb22eb5705bf30fed1ce4a14416e8fe
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-01-11 (Wed, 11 Jan 2023)

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

  Log Message:
  -----------
  [RISCV] Make sure we use LMUL=1 for scalar reduction input in combineBinOpToReduce

We might have looked through an INSERT_SUBVECTOR to find the
vmv.s.x or vfmv.s.f. If we did the ScalarV type is no longer LMUL=1.
We need to add a new INSERT_SUBVECTOR to restore it before
creating the new reduction.

While there, use the same debug location for all of the newly created
nodes. I believe we were using multiple debug locations from the
original nodes, but changing their relative order. I don't think
we're supposed to do that.




More information about the All-commits mailing list