[PATCH] D73620: Add 'gpu.terminator' operation.

[Stephan Herhut via Phabricator via llvm-commits]

herhut marked an inline comment as done.
herhut added inline comments.


================
Comment at: mlir/test/mlir-cuda-runner/shuffle.mlir:24
     store %value, %kernel_dst[%tx] : memref<?xf32>
-    gpu.return
   }
----------------
ftynse wrote:
> Nit: we may want to skip this terminator while printing, but I won't insist.
I was wondering the same. The only reason I left it printed was that I don't like a basic block without terminator. Confused my mental parser. So I only make them implicit for single-block operations. We could drop it if there is only a single block? Or is that too incoherent?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73620/new/

https://reviews.llvm.org/D73620