[PATCH] D123963: AMDGPU: Make PSV instances static members

[Jay Foad via Phabricator via llvm-commits]

foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIMachineFunctionInfo.cpp:34
   : AMDGPUMachineFunction(MF),
+    BufferPSV(static_cast<const AMDGPUTargetMachine &>(MF.getTarget())),
+    ImagePSV(static_cast<const AMDGPUTargetMachine &>(MF.getTarget())),
----------------
It doesn't look like these constructors really need an AMDGPUTargetMachine. Couldn't they all take an LLVMTargetMachine instead?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123963/new/

https://reviews.llvm.org/D123963