[PATCH] D68033: [llvm-ar] Make paths case insensitive when on windows

[Owen Reynolds via Phabricator via llvm-commits]

gbreynoo marked an inline comment as done.
gbreynoo added inline comments.


================
Comment at: llvm/test/tools/llvm-ar/windows-name-case.test:10-15
+# RUN: llvm-ar rc %t/archive.a %t/lowerCase/file.txt
+# RUN: llvm-ar rc %t/archive.a %t/UPPERCASE/FILE.TXT
+# RUN: FileCheck %s -input-file=%t/archive.a --check-prefix=ARCHIVE
+
+# ARCHIVE-NOT: file.txt
+# ARCHIVE: FILE.TXT
----------------
rupprecht wrote:
> The first archive should be checked that it doesn't contain an uppercase name (would be good to check if gnu-ar does this on windows):
> 
> ```
> # RUN: llvm-ar rc %t/archive.a %t/lowerCase/file.txt
> # RUN: FileCheck %s -input-file=%t/archive.a --check-prefix=LOWER --implicit-check-not=FILE.TXT
> # RUN: llvm-ar rc %t/archive.a %t/UPPERCASE/FILE.TXT
> # RUN: FileCheck %s -input-file=%t/archive.a --check-prefix=UPPER --implicit-check-not=file.txt
> 
> # LOWER: file.txt
> # UPPER: FILE.TXT
> ```
I've check and this is the case with gnu-ar on Windows.


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

https://reviews.llvm.org/D68033