[PATCH] D59295: [AMDGPU] Pre-allocate WWM registers to reduce VGPR pressure.

Neil Henning via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 18 07:37:10 PDT 2019


sheredom marked an inline comment as done.
sheredom added inline comments.


================
Comment at: test/CodeGen/AMDGPU/wwm-reserved.ll:13
+  %tmp109 = add i32 %tmp108, %tmp105
+  %tmp110 = tail call i32 @llvm.amdgcn.update.dpp.i32(i32 0, i32 %tmp105, i32 274, i32 15, i32 15, i1 false)
+  %tmp111 = add i32 %tmp109, %tmp110
----------------
arsenm wrote:
> Do these tests really need all of this? Can you just have a few sample instructions?
I can probably cut it down to just a single DPP inst in each WWM section - good idea!


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

https://reviews.llvm.org/D59295





More information about the llvm-commits mailing list