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

Owen Reynolds via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 07:02:39 PDT 2019


gbreynoo added a comment.

In D65743#1619508 <https://reviews.llvm.org/D65743#1619508>, @rupprecht wrote:

> 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?


mri-thin-archive.test does this but it may be best to have more cases in which the internal representation is checked.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65743





More information about the llvm-commits mailing list