[PATCH] D85517: [Scheduling] Implement a new way to cluster loads/stores

Tue Aug 25 02:24:24 PDT 2020

foad accepted this revision.
foad added a comment.
This revision is now accepted and ready to land.

In D85517#2206517 <https://reviews.llvm.org/D85517#2206517>, @foad wrote:

> I've tested this patch by compiling several thousand shaders with the AMDGPU backend and analysing the generated code. It gives a nice ~ 4% increase in the average size of a load cluster (or equivalently a ~ 4% decrease in the total number of clusters, given that the total number of loads is unchanged).

I've tested this again and it now gives a ~ 0.25% increase in the amount of clustering. LGTM.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85517