[PATCH] D68146: [FileCheck] Implement --ignore-case option.
Jordan Rupprecht via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 10 10:13:35 PDT 2019
rupprecht added a comment.
In D68146#1703693 <https://reviews.llvm.org/D68146#1703693>, @thakis wrote:
> The test fails on Linux: http://lab.llvm.org:8011/builders/clang-cmake-x86_64-sde-avx512-linux/builds/28537/steps/ninja%20check%201/logs/FAIL%3A%20LLVM%3A%3Acheck-ignore-case.txt If it takes a while to fix please revert while you investigate.
>
> Also, https://github.com/llvm/llvm-project/commit/dfd2b6f07fc40a190335f580d8a965bbebfe94df looks like you touched ~all lines in docs/CommandGuide/FileCheck.rst and llvm/include/llvm/Support/FileCheck.h Maybe you converted them to windows line endings? If so, please undo that. (Maybe revert and reland with fixed line endings so that the diff for the actual change is readable.)
As described in http://llvm.org/docs/GettingStarted.html#checkout-llvm-from-git, the right way to checkout the repository on windows is:
% git clone --config core.autocrlf=false https://github.com/llvm/llvm-project.git
This is the second time I've reviewed a change like this without realizing, would appreciate tips if this could be more visible in Phab somehow...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D68146/new/
https://reviews.llvm.org/D68146
More information about the llvm-commits
mailing list