[PATCH] D111646: [AMDGPU] Enable load clustering in the post-RA scheduler

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

In D111646#3063346 <https://reviews.llvm.org/D111646#3063346>, @piotr wrote:

> Sounds good to me. The runtime improvement from clustering is notoriously difficult to assess, but your static data shows some potential benefit.
>
> Having said that, it probably makes sense to guard the mutation by the optlevel check, so we only enable it with -O2 or higher.

Interesting. I wonder if we should do that for all the mutations, or even all the (post-ra?) scheduling?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111646