[PATCH] D69343: [llvm-ar] Allow MRI scripts to be passed via command line argument

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

Honestly I'd try to avoid adding a new command line option to llvm-ar. Especially, in this case there are several easy workaround as follows:

- instead of passing contents of MRI script as an external file you can use shell redirection
- if shell is not available or for some reason you don't want to use it, you can write a wrapper that passes an MRI script to llvm-ar

Also, adding an incompatible new feature to llvm-ar will hurt the interoperability with other ar commands. Once you add a new feature and depend on it, you can't go back to the other ar commands.

Compared to the above points, the benefit of doing this is not very clear to me. If you have more contexts, could you elaborate?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D69343