[PATCH] D76737: [mlir][vulkan-runner] Add support for 2D memref.

Lei Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 26 12:33:07 PDT 2020


antiagainst accepted this revision.
antiagainst added a comment.
This revision is now accepted and ready to land.

Nice, thanks!



================
Comment at: mlir/lib/Conversion/GPUToVulkan/ConvertLaunchFuncToVulkanCalls.cpp:102
+    // Create a type `!llvm<"{ float*, float*, i64, [2 x i64], [2 x i64]}">`.
+    llvmMemRef2DFloat = LLVM::LLVMType::getStructTy(
+        llvmDialect,
----------------
This is similar enough with the above; but we only have two instance here so I'm fine with it atm. Might want to dedup by having a function later if there are more cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76737





More information about the llvm-commits mailing list