[PATCH] D57927: [llvm-ar] Implement the P modifier.

[Rui Ueyama via Phabricator via llvm-commits]

ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM

Hmm, that's right. That's probably a good reason why we can't make P the default behavior.

ar doesn't store directory names to an archive file, but it tries to hide that fact from the user in some degree, but the mismatch leaks out in many situations like the above. Probably, ar shouldn't have tried to do that from the beginning, but we are perhaps 50+ years too late to fix it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D57927