[PATCH] D59491: Fix relative thin archive path handling

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 17 19:27:04 PDT 2019


ruiu added a comment.

Sorry, I though that I replied to this thread.

In D59491#1436706 <https://reviews.llvm.org/D59491#1436706>, @gbreynoo wrote:

> The problem with working out the paths is if either are in the form `foo/../bar/`. I don't believe your above suggestion would work for paths with internal `../`.


You can just call remove_dots to remove all internal `..`, no?


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

https://reviews.llvm.org/D59491





More information about the llvm-commits mailing list