================
@@ -183,7 +187,9 @@ llvm::Error 
EntityLinker::link(std::unique_ptr<TUSummaryEncoding> Summary) {
 
   auto EntityResolutionTable = resolve(SummaryRef);
   auto PatchTargets = merge(SummaryRef, EntityResolutionTable);
-  patch(PatchTargets, EntityResolutionTable);
+  if (auto Err = patch(PatchTargets, EntityResolutionTable)) {
+    return Err;
+  }
 
   return llvm::Error::success();
----------------
steakhal wrote:

Shall we just have `return patch(...);` directly?

https://github.com/llvm/llvm-project/pull/184713
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to