[Mlir-commits] [mlir] [mlir][gpu] Add address space modifier to Barrier (PR #110527)

Fabian Mora llvmlistbot at llvm.org
Mon Sep 30 10:18:19 PDT 2024


fabianmcg wrote:

> @fabianmcg I would say more accurately that MLIR's `gpu.barrier` is currently defined to include a memory fence, so this is a proposed weakening

I agree, thread barriers including `gpu.barrier` usually imply memory fences, however, their semantics are stronger and people are expecting that by definition.

That's why I think a `memfence` operation might be better, as it makes clear (`barrier` and `memfence`) have different semantics.

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


More information about the Mlir-commits mailing list