[PATCH] D56508: [llvm-ar] Flatten thin archives.

[Jordan Rupprecht via Phabricator via llvm-commits]

rupprecht added a comment.

In D56508#1397156 <https://reviews.llvm.org/D56508#1397156>, @void wrote:

> This fails if the archives are nested. E.g.:
>
>   $ mkdir -p foo/bar
>   $ cat > foo/bar/a.c <<EOF
>   int bar() { return 42; }
>   EOF
>   $ rm -f built-in.o foo/built-in.o foo/bar/built-in.o ; llvm-ar rcSTD foo/bar/built-in.o foo/bar/a.o ; llvm-ar rcSTD foo/built-in.o foo/bar/built-in.o ; llvm-ar rcSTD built-in.o foo/built-in.o
>   ./llvm.opt.install/bin/llvm-ar: error: 'a.o': No such file or directory.
>
>
> What's happening is that it's trying to find `a.o` in `foo/` when it's really in `foo/bar/`.


That should be fixed by D57842 <https://reviews.llvm.org/D57842>, which I'm planning to submit soon -- I want to poke a few more test cases first.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56508