[PATCH] D59491: Fix relative thin archive path handling

Owen Reynolds via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 04:04:49 PDT 2019


gbreynoo updated this revision to Diff 201186.
gbreynoo added a comment.
Herald added subscribers: MaskRay, jakehehrlich, arichardson, emaste.
Herald added a reviewer: espindola.
Herald added a reviewer: alexshap.
Herald added a reviewer: jhenderson.

I've attempted to make "computeArchiveRelativePath" clearer and errors are now propagated to the caller to deal with rather than just returning the "To" argument.


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

https://reviews.llvm.org/D59491

Files:
  include/llvm/Object/ArchiveWriter.h
  lib/Object/ArchiveWriter.cpp
  lib/ToolDrivers/llvm-lib/LibDriver.cpp
  test/tools/llvm-ar/reduce-thin-path.test
  test/tools/llvm-ar/thin-archive.test
  test/tools/llvm-objcopy/ELF/archive-unknown-members.test
  test/tools/llvm-readobj/thin-archive-paths.test
  tools/llvm-ar/llvm-ar.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59491.201186.patch
Type: text/x-patch
Size: 11701 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190524/c4d3a802/attachment.bin>


More information about the llvm-commits mailing list