[all-commits] [llvm/llvm-project] d6f942: GlobalISel: Pass MachineIRBuilder to applyMappingImpl

Matt Arsenault via All-commits all-commits at lists.llvm.org
Mon Jul 31 07:03:53 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d6f9428e46afbeeaedf2727682fa413381953511
      https://github.com/llvm/llvm-project/commit/d6f9428e46afbeeaedf2727682fa413381953511
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2023-07-31 (Mon, 31 Jul 2023)

  Changed paths:
    M llvm/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h
    M llvm/include/llvm/CodeGen/RegisterBankInfo.h
    M llvm/lib/CodeGen/GlobalISel/RegBankSelect.cpp
    M llvm/lib/Target/AArch64/GISel/AArch64RegisterBankInfo.cpp
    M llvm/lib/Target/AArch64/GISel/AArch64RegisterBankInfo.h
    M llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.cpp
    M llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.h
    M llvm/lib/Target/Mips/MipsRegisterBankInfo.cpp
    M llvm/lib/Target/Mips/MipsRegisterBankInfo.h
    M llvm/lib/Target/X86/X86RegisterBankInfo.cpp
    M llvm/lib/Target/X86/X86RegisterBankInfo.h

  Log Message:
  -----------
  GlobalISel: Pass MachineIRBuilder to applyMappingImpl

The target should not have to construct MachineIRBuilders during
RegBankSelect (we should perhaps hide the constructors for it). The
pass should own the builder setup with the desired CSE configuration
(although currently the pass does not use the CSE builder, which is
what I want to fix).

https://reviews.llvm.org/D156479




More information about the All-commits mailing list