[PATCH] D81109: llvm-link: Add support for archive files as inputs.

Jan Sjödin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 12 14:20:07 PDT 2020


jsjodin marked 2 inline comments as done.
jsjodin added inline comments.


================
Comment at: llvm/tools/llvm-link/llvm-link.cpp:207
+  } // end for each child
+  ExitOnErr(std::move(Err));
+  return Result;
----------------
ABataev wrote:
> Does it really worth using `std::move` here and in other places?
Yes, otherwise you get a compilation error.


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

https://reviews.llvm.org/D81109





More information about the llvm-commits mailing list