[all-commits] [llvm/llvm-project] b95d50: Add and call `AMDGPUMCResourceInfo::reset` method ...

Thomas Symalla via All-commits all-commits at lists.llvm.org
Wed Oct 2 05:17:23 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b95d50e5d877e2a59d24f8242eb85c92a4c29ab0
      https://github.com/llvm/llvm-project/commit/b95d50e5d877e2a59d24f8242eb85c92a4c29ab0
  Author: Thomas Symalla <5754458+tsymalla at users.noreply.github.com>
  Date:   2024-10-02 (Wed, 02 Oct 2024)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPUAsmPrinter.cpp
    M llvm/lib/Target/AMDGPU/AMDGPUMCResourceInfo.cpp
    M llvm/lib/Target/AMDGPU/AMDGPUMCResourceInfo.h

  Log Message:
  -----------
  Add and call `AMDGPUMCResourceInfo::reset` method (#110818)

When compiling multiple pipelines, the `MCRegisterInfo` instance in
`AMDGPUAsmPrinter` gets re-used even after finalization, so it calls
`finalize()` multiple times.

Add a reset method and call it in
`AMDGPUAsmPrinter::doFinalization`.

Different approach would be to make it a `unique_ptr`.

---------

Co-authored-by: Thomas Symalla <tsymalla at amd.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list