[Mlir-commits] [mlir] [mlir][ptr] Add high-level read and write memory operations (PR #161081)
Fabian Mora
llvmlistbot at llvm.org
Tue Sep 30 04:53:51 PDT 2025
fabianmcg wrote:
> Even the naming looks out of place: how would one figure why we can "load" and "read" from a pointer? The names looks synonymous and just arbitrarily introduced "just because".
FWIW, I took the naming from `vector`, where there are `load`, `store` and `transfer_read`, `transfer_write` ops.
I can change the names of these ops to `transfer_read`, `transfer_write`, or find another suitable name.
> This just does not belong to `ptr` IMO: let's keep things simple and generalizable by avoiding ad-hoc constructions.
These are still ops acting on pointers with well-defined semantics in the dialect, so I wouldn't call them ad-hoc constructions. However, I can give you that this does introduce 2 different semantic levels in the dialect, and while there are other dialects that do that (eg. `vector`), it still might not be desirable.
https://github.com/llvm/llvm-project/pull/161081
More information about the Mlir-commits
mailing list