[PATCH] D123343: [AMDGPU] Refactor LDS alignment checks.

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added a comment.

In D123343#3448518 <https://reviews.llvm.org/D123343#3448518>, @rampitec wrote:

> I have a w/a for this in the D123634 <https://reviews.llvm.org/D123634>, but in general I do not think 'fast' or 'slow' is a right measure. Something is not fast or slow, but faster or slower than something other. This would be a big change though.

Speaking of fast and slow this might be modeled with an unsigned speed rank. The direct translation of current bool IsFast will be to 0 and 1. Then target may use more ranks and a higher the number the faster is the access.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123343