[all-commits] [llvm/llvm-project] d95b82: [NFC][AMDGPU] Make AMDGPUSplitModule a ModulePass ...

[Pierre van Houtryve via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d95b82c49aef0223bcc03ff5a9163e70037c82be
      https://github.com/llvm/llvm-project/commit/d95b82c49aef0223bcc03ff5a9163e70037c82be
  Author: Pierre van Houtryve <pierre.vanhoutryve at amd.com>
  Date:   2024-06-18 (Tue, 18 Jun 2024)

  Changed paths:
    M llvm/include/llvm/Target/TargetMachine.h
    M llvm/lib/Target/AMDGPU/AMDGPUSplitModule.cpp
    M llvm/lib/Target/AMDGPU/AMDGPUSplitModule.h
    M llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
    M llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.h

  Log Message:
  -----------
  [NFC][AMDGPU] Make AMDGPUSplitModule a ModulePass (#95773)

It allows it to access TTI correctly, and opens the door to accessing
more analysis in the future.

I went back and forth between this, and also making the default
SplitModule a Pass too to make it uniform, but I decided against it
because it's just needless complications. Neither llvm-split or
LTOBackend have a PM ready to use so we need to create one anyway. Let's
keep all the mess hidden in the AMDGPU version for now to keep this
change more self-contained.



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