[Mlir-commits] [mlir] [mlir][spirv] Do SPIR-V serialization in -test-vulkan-runner-pipeline (PR #121494)

[Jakub Kuderski]

================
@@ -135,21 +132,33 @@ LogicalResult ConvertGpuLaunchFuncToVulkanLaunchFunc::declareVulkanLaunchFunc(
   return success();
 }
 
-LogicalResult ConvertGpuLaunchFuncToVulkanLaunchFunc::createBinaryShader(
-    ModuleOp module, std::vector<char> &binaryShader) {
+LogicalResult ConvertGpuLaunchFuncToVulkanLaunchFunc::getBinaryShader(
+    ModuleOp module, StringAttr &binaryAttr) {
----------------
kuhar wrote:

Instead of the out parameter, you can return `FailureOr<StringAttr>`

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