[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