[PATCH] D65743: [llvm-ar] Accept file paths with windows format slashes

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 7 11:45:34 PDT 2019


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

I think it'd be good to also write a test that verifies the internal representation is always forward slashes on windows by running `FileCheck` directly on the thin archive instead of using `llvm-ar t` to look at it (e.g. see thin-archive.test). Or do we already have a test for that?


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

https://reviews.llvm.org/D65743





More information about the llvm-commits mailing list