[PATCH] D122850: [AMDGPU] Fix regression with vectorization limiting

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 5 10:25:09 PDT 2022


rampitec added a comment.

In D122850#3430133 <https://reviews.llvm.org/D122850#3430133>, @dfukalov wrote:

> It seems to me `Cost::RateFormula()` form LSR is the only user that can be affected by the change. Would you please double-look the use case?

Yes, I believe unbound LSR also hits us with RP a lot. Limiting it to just 4 'free' pointers is a good thing IMO.


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

https://reviews.llvm.org/D122850



More information about the llvm-commits mailing list