[Mlir-commits] [mlir] [mlir][ExecutionEngine] propagate errors in mlirExecutionEngineCreate (PR #170592)

[llvmlistbot at llvm.org]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir-execution-engine

Author: Maksim Levental (makslevental)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/170592.diff


1 Files Affected:

- (modified) mlir/lib/CAPI/ExecutionEngine/ExecutionEngine.cpp (+7-3) 


``````````diff
diff --git a/mlir/lib/CAPI/ExecutionEngine/ExecutionEngine.cpp b/mlir/lib/CAPI/ExecutionEngine/ExecutionEngine.cpp
index 2dbb993b1640f..ab53657577aec 100644
--- a/mlir/lib/CAPI/ExecutionEngine/ExecutionEngine.cpp
+++ b/mlir/lib/CAPI/ExecutionEngine/ExecutionEngine.cpp
@@ -38,12 +38,15 @@ mlirExecutionEngineCreate(MlirModule op, int optLevel, int numPaths,
 
   auto tmBuilderOrError = llvm::orc::JITTargetMachineBuilder::detectHost();
   if (!tmBuilderOrError) {
-    llvm::errs() << "Failed to create a JITTargetMachineBuilder for the host\n";
+    llvm::errs()
+        << "Failed to create a JITTargetMachineBuilder for the host because: "
+        << tmBuilderOrError.takeError() << "\n";
     return MlirExecutionEngine{nullptr};
   }
   auto tmOrError = tmBuilderOrError->createTargetMachine();
   if (!tmOrError) {
-    llvm::errs() << "Failed to create a TargetMachine for the host\n";
+    llvm::errs() << "Failed to create a TargetMachine for the host because: "
+                 << tmOrError.takeError() << "\n";
     return MlirExecutionEngine{nullptr};
   }
 
@@ -62,7 +65,8 @@ mlirExecutionEngineCreate(MlirModule op, int optLevel, int numPaths,
   jitOptions.enableObjectDump = enableObjectDump;
   auto jitOrError = ExecutionEngine::create(unwrap(op), jitOptions);
   if (!jitOrError) {
-    consumeError(jitOrError.takeError());
+    llvm::errs() << "Failed to create an ExecutionEngine because: "
+                 << jitOrError.takeError() << "\n";
     return MlirExecutionEngine{nullptr};
   }
   return wrap(jitOrError->release());

``````````

</details>


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