[all-commits] [llvm/llvm-project] baa107: [RISCV] Only expose one interface for getContainer...

Craig Topper via All-commits all-commits at lists.llvm.org
Fri Apr 23 15:20:17 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: baa107f018a94763d829daa534b4b1bf98219306
      https://github.com/llvm/llvm-project/commit/baa107f018a94763d829daa534b4b1bf98219306
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2021-04-23 (Fri, 23 Apr 2021)

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

  Log Message:
  -----------
  [RISCV] Only expose one interface for getContainerForFixedLengthVector in the RISCVTargetLowering class

We can have RISCVISelDAGToDAG.cpp call the VT only version by
finding the RISCVTargetLowering object via the Subtarget.

Make the static versions just global static functions in
RISCVISelLowering that can be called by static functions in that
file.


  Commit: bcf321015b10b02dc1e9f6a67017713344501610
      https://github.com/llvm/llvm-project/commit/bcf321015b10b02dc1e9f6a67017713344501610
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2021-04-23 (Fri, 23 Apr 2021)

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

  Log Message:
  -----------
  [RISCV] Move getLMULForFixedLengthVector out of RISCVSubtarget.

Make it a static function RISCVISelLowering, the only place it
is used.

I think I'm going to make this return a fractional LMULs in some
cases so I'm sorting out where it should live before I start
making changes.


Compare: https://github.com/llvm/llvm-project/compare/3fe5c3b0189f...bcf321015b10


More information about the All-commits mailing list