[PATCH] D37134: [libc++] Use CMAKE_AR for merging static libraries

[Eric Fiselier via Phabricator via cfe-commits]

EricWF accepted this revision.
EricWF added a comment.

LGTM, minus inline comments.



================
Comment at: utils/merge_archives.py:97
     parser.add_argument(
+        '-a', '--ar', dest='ar_exe', required=False,
+        help='The ar executable to use, finds \'ar\' in the path if not given',
----------------
`-a` doesn't feel very descriptive, and unlike `-o` and `-L` it isn't used commonly elsewhere. Maybe just the long option on this one?


https://reviews.llvm.org/D37134