[PATCH] D157599: [AMDGPU] Do not release VGPRs at -O0

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 10 03:50:34 PDT 2023


foad added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp:1039
     if (ST->getGeneration() >= AMDGPUSubtarget::GFX11 &&
+        MI.getMF()->getTarget().getOptLevel() != CodeGenOpt::None &&
         ScoreBrackets.getScoreRange(VS_CNT) != 0 &&
----------------
arsenm wrote:
> should also account for optnone on the function
Is there a helper I can use that takes both of these into account? In the whole of the backend I only see one explicit check for F.hasOptNone(), in AMDGPURegBankSelect.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157599



More information about the llvm-commits mailing list