Revert "[AMDGPU] Do not assume stack size for PAL code object indirect calls"

This reverts commit 3604fdf.

It is causing CTS failures with DebugOverlayEnabled including:
dEQP-VK.binding_model.descriptor_update.acceleration_structure.ray_query.regular.ahit

Change-Id: Ib30e86fdb4d901865778f58448c8147d067897fc

Author: jayfoad

llvm/lib/Target/AMDGPU/AMDGPUResourceUsageAnalysis.cpp

@@ -104,16 +104,14 @@ bool AMDGPUResourceUsageAnalysis::runOnModule(Module &M) {
 
   MachineModuleInfo &MMI = getAnalysis<MachineModuleInfoWrapperPass>().getMMI();
   const TargetMachine &TM = TPC->getTM<TargetMachine>();
-  const MCSubtargetInfo &STI = *TM.getMCSubtargetInfo();
   bool HasIndirectCall = false;
 
   CallGraph CG = CallGraph(M);
   auto End = po_end(&CG);
 
   // By default, for code object v5 and later, track only the minimum scratch
   // size
-  if (AMDGPU::getCodeObjectVersion(M) >= AMDGPU::AMDHSA_COV5 ||
-      STI.getTargetTriple().getOS() == Triple::AMDPAL) {
+  if (AMDGPU::getCodeObjectVersion(M) >= AMDGPU::AMDHSA_COV5) {
     if (!AssumedStackSizeForDynamicSizeObjects.getNumOccurrences())
       AssumedStackSizeForDynamicSizeObjects = 0;
     if (!AssumedStackSizeForExternalCall.getNumOccurrences())