[PATCH] D95610: [AMDGPU] Clarify calling conv about inactive lanes

[Tony Tye via Phabricator via llvm-commits]

t-tye added inline comments.


================
Comment at: llvm/docs/AMDGPUUsage.rst:8263
+
+    * Lanes of all VGPRs that are inactive at the call site.
 
----------------
madhur13490 wrote:
> I am confused by the wording "Lanes of all VGPRs". The word "lane" is used in the context of workitems and not VGPRs. Do you mean to say - "All VGPRs of inactive lanes at the call site"?
One view of VGPRs is that that comprises a separate dword for each lane. This sentence is stating that the VGPR lanes corresponding to inactive lanes must be preserved.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95610