[PATCH] D43987: Fix ar command line expansion on Windows.

[Michael Spencer via Phabricator via llvm-commits]

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

This is good, but this should be using llvm-ar's `failIfError` instead of adding `ExitOnError`. Moving the error handling to `ExitOnError` should be a separate commit if done.

Really all of our tools should be doing this, as it also (mostly) properly handles unicode paths on Windows.


https://reviews.llvm.org/D43987