[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