[PATCH] D115559: AMDGPU: Propagate amdgpu-waves-per-eu with attributor

Joseph Huber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 17:30:09 PDT 2023


jhuber6 added a comment.

In D115559#4407258 <https://reviews.llvm.org/D115559#4407258>, @arsenm wrote:

> In D115559#4396025 <https://reviews.llvm.org/D115559#4396025>, @jhuber6 wrote:
>
>> There is actually an assumption propagation AA in the Attributor already https://github.com/llvm/llvm-project/blob/main/llvm/lib/Transforms/IPO/AttributorAttributes.cpp#L11609. I don't know if that's relevant to what the AMDGPUAttributor wants to do however.
>
> Don't see how assumes are involved. There isn't a way to introspect waves-per-eu

I was just wondering out loud if you could model this as "assume one thread" and then treat it the same way in the backend.


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

https://reviews.llvm.org/D115559



More information about the llvm-commits mailing list