[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