This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Adding gpu.host_register op and lower it to a runtime call.
ClosedPublic

Authored by csigg on Aug 10 2020, 1:14 AM.

Diff Detail

Event Timeline

csigg created this revision.Aug 10 2020, 1:14 AM
Herald added a reviewer: ftynse. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
csigg requested review of this revision.Aug 10 2020, 1:14 AM
herhut accepted this revision.Aug 10 2020, 10:59 AM

Thanks!

mlir/include/mlir/Conversion/StandardToLLVM/ConvertStandardToLLVM.h
444

Nice!

mlir/include/mlir/Dialect/GPU/GPUOps.td
745

You can drop the Results

mlir/tools/mlir-rocm-runner/rocm-runtime-wrappers.cpp
79–87

nit: CUDA -> ROCm

This revision is now accepted and ready to land.Aug 10 2020, 10:59 AM
csigg updated this revision to Diff 284479.Aug 10 2020, 12:39 PM
csigg marked an inline comment as done.

Apply reviewer comments.

csigg marked an inline comment as done.Aug 10 2020, 12:39 PM

Thanks a lot for the review, Stephan.

This revision was landed with ongoing or failed builds.Aug 10 2020, 1:46 PM
This revision was automatically updated to reflect the committed changes.

(just two nits)

mlir/include/mlir/Dialect/GPU/GPUOps.td
749

Can you extend the description, it is quite terse for something that isn't that trivial I think.

In particular I think this should explain is this is just memory space mapping and what is the expected consistency model in terms of visibility / fences.

mlir/lib/Conversion/GPUCommon/ConvertLaunchFuncToRuntimeCalls.cpp
128

s/patter/pattern/