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

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 6 23:28:41 PST 2019


ruiu added a comment.

In D69418#1735511 <https://reviews.llvm.org/D69418#1735511>, @kongyi wrote:

> I'll create a separate patch to address the argument ordering comment.


It is not clear to me whether my concern has already been addressed (and only testcase is missing) or is not implemented. Which is the case? If it's not implemented, maybe we should do that as part of this patch to reduce the risk of getting this feature in to the next release without the safeguard?


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

https://reviews.llvm.org/D69418





More information about the llvm-commits mailing list