[PATCH] D69418: [llvm-ar] Add output option for extract operation

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 25 09:48:10 PDT 2019


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

In D69418#1721179 <https://reviews.llvm.org/D69418#1721179>, @ruiu wrote:

> I think I'm leaning toward not adding this option at the moment for the compatibility reason. IIUC, llvm-ar was created because otherwise it would have been hard to create archives containing symbols from bitcode files, and except that feature, it was intended to be a drop-in replacement. If you land this patch and start depending on the feature, you can't easily go back to the other ar commands.
>
> If llvm-ar is the most popular ar command, we can make a new option de-fact standard, but unfortunately that's not the case.
>
> This option doesn't really depend on LLVM. If this feature is useful, this would be useful for other ar commands, and adding this option to a major ar command would maximizes the benefit. Maybe you should propose this option to GNU first?


I get a positive response from Nick Clifton https://sourceware.org/ml/binutils/2019-10/msg00186.html , so it is likely the option can be added to GNU ar as well. I'll have to figure out how to implement that, though...


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

https://reviews.llvm.org/D69418





More information about the llvm-commits mailing list