[PATCH] D98362: [AMDGPU] Fix -amdgpu-inline-arg-alloca-cost
    Stanislav Mekhanoshin via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Mar 10 11:57:21 PST 2021
    
    
  
rampitec added inline comments.
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp:1192
   if (AllocaSize)
-    return ArgAllocaCost;
+    return ArgAllocaCost * getInliningThresholdMultiplier();
   return 0;
----------------
arsenm wrote:
> Should the just adjust for the scale then?
I thought about this, but whenever we will adjust the scale the next time we will have to visit it again.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98362/new/
https://reviews.llvm.org/D98362
    
    
More information about the llvm-commits
mailing list