[llvm] [AMDGPU] SIInsertHardClause: add configurable clause length limit (PR #142343)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 2 02:50:35 PDT 2025
jayfoad wrote:
> > Why?
>
> In certain applications I am seeing large clauses to have a negative impact on performance. For these cases using small clauses (e.g. 4 operations) boosts performance 1-2%. (Disabling clausing entirely does not yield this benefit.) I assume these cases have improved cache efficiency from interleaving lock-stepped waves rather than issuing long sequences of uninterrupted memory requests from each wave in turn.
>
> I am gather data to see if a lower (than hardware maximum) limit might be beneficial overall.
Fair enough. Should mention in the commit description that it's for performance tuning.
https://github.com/llvm/llvm-project/pull/142343
More information about the llvm-commits
mailing list