[llvm-branch-commits] [llvm] AMDGPU: Fix buffer load/store of pointers (PR #95379)

Krzysztof Drewniak via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu Jun 13 07:35:50 PDT 2024


krzysz00 wrote:

So, general question on this patch series:

Wouldn't it be more reasonable to, instead of having separate handling for all the possible register types, always do loads as `i8`, `i16`, `i32` `<2 x i32>`, `<3 x i32>, or `<4 x i32>` and then `bitcast`/`merge_values`/... the results back to their type?

Or at least to have that fallback path - if we don't know what a type is, load/store it as its bits?

(Then we wouldn't need to, for example, go back and add a `<16 x i8>` case if someone realizes they want that)

https://github.com/llvm/llvm-project/pull/95379


More information about the llvm-branch-commits mailing list