[all-commits] [llvm/llvm-project] 3e6069: [RISCV][GISel] Rename PartialMappingIdx and ValueM...

Craig Topper via All-commits all-commits at lists.llvm.org
Sun Nov 12 00:09:30 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3e60693422cc10de5b593d6bd532a82ceb9fac9c
      https://github.com/llvm/llvm-project/commit/3e60693422cc10de5b593d6bd532a82ceb9fac9c
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/GISel/RISCVRegisterBankInfo.cpp

  Log Message:
  -----------
  [RISCV][GISel] Rename PartialMappingIdx and ValueMappingsIdx to use register bank name with B suffix. NFC

PartialMappingIdx refers to an entry in the PartMappings table that
includes a RegBank and a size. So I think the PartialMappingIdx should
be named using the name of the register bank concatenated with the size.

So the indices are now PMI_GPRB32, PMI_GPRB64, PMI_FPRB32, PMI_FPRB64.


  Commit: 1ffcac1b077afecff4b57b2ceec73c9bcb847389
      https://github.com/llvm/llvm-project/commit/1ffcac1b077afecff4b57b2ceec73c9bcb847389
  Author: Craig Topper <craig.topper at sifive.com>
  Date:   2023-11-12 (Sun, 12 Nov 2023)

  Changed paths:
    M llvm/lib/Target/RISCV/GISel/RISCVRegisterBankInfo.cpp

  Log Message:
  -----------
  [RISCV][GISel] Rename ValueMappingsIdx->ValueMappingIdx. NFC

This is consistent with other targets.


Compare: https://github.com/llvm/llvm-project/compare/8c53cfd351a9...1ffcac1b077a


More information about the All-commits mailing list