AMDGPU: Use caller subtarget, not intrinsic declaration

[arsenm]

Summary

Intrinsic declarations use the default subtarget, but this should be
using the subtarget for the calling function. I haven't been able to
come up with a case where it matters though.

[rampitec]

LGTM

[arsenm]

a1bc37c9e54e0163bc6ccb7a438a68047310ccff

llvm/lib/Target/AMDGPU/AMDGPULowerIntrinsics.cpp

bool AMDGPULowerIntrinsics::makeLIDRangeMetadata(Function &F) const {
   const TargetMachine &TM = TPC->getTM<TargetMachine>();
   bool Changed = false;
 
   for (auto *U : F.users()) {
     auto *CI = dyn_cast<CallInst>(U);
     if (!CI)
       continue;
 
-    Changed |= AMDGPUSubtarget::get(TM, F).makeLIDRangeMetadata(CI);
+    Function *Caller = CI->getParent()->getParent();
+    const AMDGPUSubtarget &ST = AMDGPUSubtarget::get(TM, *Caller);
+    Changed |= ST.makeLIDRangeMetadata(CI);
   }
   return Changed;
 }
 
 bool AMDGPULowerIntrinsics::runOnModule(Module &M) {
   bool Changed = false;
 
   for (Function &F : M) {

llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h

public:
   /// \returns Maximum number of waves per execution unit supported by the
   /// subtarget without any kind of limitation.
   unsigned getMaxWavesPerEU() const { return MaxWavesPerEU; }
 
   /// Return the maximum workitem ID value in the function, for the given (0, 1,
   /// 2) dimension.
   unsigned getMaxWorkitemID(const Function &Kernel, unsigned Dimension) const;
 
-  /// Creates value range metadata on an workitemid.* inrinsic call or load.
+  /// Creates value range metadata on an workitemid.* intrinsic call or load.
   bool makeLIDRangeMetadata(Instruction *I) const;
 
   /// \returns Number of bytes of arguments that are passed to a shader or
   /// kernel in addition to the explicit ones declared for the function.
   unsigned getImplicitArgNumBytes(const Function &F) const {
     if (isMesaKernel(F))
       return 16;
     return AMDGPU::getIntegerAttribute(F, "amdgpu-implicitarg-num-bytes", 0);