[PATCH] D95762: AMDGPU: Don't consider global pressure when bundling soft clauses

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 31 08:16:18 PST 2021


arsenm created this revision.
arsenm added a reviewer: rampitec.
Herald added subscribers: kerbowa, hiraditya, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl.
arsenm requested review of this revision.
Herald added a subscriber: wdng.
Herald added a project: LLVM.

This should only consider whether the pressure impact of the relevant
at the given point in the program will decrease the occupancy. High
VGPR pressure was incorrectly blocking the formation of scalar
bundles, and vice versa. This was also blocking bundling from high
pressure situations at other points in the program.


https://reviews.llvm.org/D95762

Files:
  llvm/lib/Target/AMDGPU/GCNRegPressure.h
  llvm/lib/Target/AMDGPU/SIFormMemoryClauses.cpp
  llvm/test/CodeGen/AMDGPU/soft-clause-bundle-local-pressure.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95762.320358.patch
Type: text/x-patch
Size: 14900 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210131/8b9bfc18/attachment.bin>


More information about the llvm-commits mailing list