[clang] [clang][lex] Fix non-portability diagnostics with absolute path (PR #74782)

2024-01-02 Thread Jeremy Wong via cfe-commits 

jrmwng wrote:

492>Failed Tests (1):
492>  Clang :: Lexer/case-insensitive-include-absolute.c

I built with "clang", "X86" and "AArch64" under Windows 11 and VS2022 community 
edition

```
492>-- Build started: Project: check-all, Configuration: Release x64 --
492>Running all regression tests
492>llvm-lit.py: C:\GitHub\llvm-project\llvm\utils\lit\lit\llvm\config.py:46: 
note: using lit tools: C:\Program Files\Git\usr\bin
492>llvm-lit.py: C:\GitHub\llvm-project\llvm\utils\lit\lit\llvm\config.py:488: 
note: using clang: c:\github\llvm-project\build\release\bin\clang.exe
492>-- Testing: 70944 tests, 32 workers --
492>FAIL: Clang :: Lexer/case-insensitive-include-absolute.c (1 of 70944)
492> TEST 'Clang :: 
Lexer/case-insensitive-include-absolute.c' FAILED 
492>Exit Code: 2
492>
492>Command Output (stdout):
492>--
492># RUN: at line 3
492>rm -rf 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp
 && split-file 
C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp
492># executed command: rm -rf 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp'
492># executed command: split-file 
'C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c' 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp'
492># RUN: at line 4
492>sed 
"s|DIR|C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp|g"
 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c.in
 > 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c
492># executed command: sed 
's|DIR|C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp|g'
 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c.in'
492># RUN: at line 5
492>c:\github\llvm-project\build\release\bin\clang.exe -cc1 -internal-isystem 
F:\GitHub\llvm-project\build\Release\lib\clang\18\include -nostdsysteminc 
-fsyntax-only 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c
 2>&1 | c:\github\llvm-project\build\release\bin\filecheck.exe 
C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c 
--DDIR=C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp
492># executed command: 'c:\github\llvm-project\build\release\bin\clang.exe' 
-cc1 -internal-isystem 
'F:\GitHub\llvm-project\build\Release\lib\clang\18\include' -nostdsysteminc 
-fsyntax-only 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c'
492># executed command: 
'c:\github\llvm-project\build\release\bin\filecheck.exe' 
'C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c' 
--DDIR=C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp
492># .---command stderr
492>CUSTOMBUILD : # | FileCheck error : '' is empty.
492># | FileCheck command line:  
c:\github\llvm-project\build\release\bin\filecheck.exe 
C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c 
--DDIR=C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp
492># `-
492>CUSTOMBUILD : # error : command failed with exit status: 2
492>
492>--
492>
492>
492>
492>Failed Tests (1):
492>  Clang :: Lexer/case-insensitive-include-absolute.c
492>
492>
492>Testing Time: 434.26s
492>
492>Total Discovered Tests: 92696
492>  Skipped  :90 (0.10%)
492>  Unsupported  : 24219 (26.13%)
492>  Passed   : 68283 (73.66%)
492>  Expectedly Failed:   103 (0.11%)
492>  Failed   : 1 (0.00%)
492>C:\Program Files\Microsoft Visual 
Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5):
 error MSB8066: Custom build for 
'C:\GitHub\llvm-project\build\CMakeFiles\5344a242bf0d5667ee3e0d3564612537\check-all.rule;C:\GitHub\llvm-project\llvm\CMakeLists.txt'
 exited with code 1.
492>Done building project "check-all.vcxproj" -- FAILED.
== Build: 491 succeeded, 1 failed, 55 up-to-date, 0 skipped ==
```

https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][lex] Fix non-portability diagnostics with absolute path (PR #74782)

2024-01-04 Thread Jeremy Wong via cfe-commits 

jrmwng wrote:

@jansvoboda11 One thing is special.

The path ``C:\GitHub`` is a symbolic link to another harddisk.

```
 GitHub [F:\GitHub]
```

https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][lex] Fix non-portability diagnostics with absolute path (PR #74782)

2024-01-04 Thread Jeremy Wong via cfe-commits 

jrmwng wrote:

@jansvoboda11 it should be the following steps. (I try to repeat it now)

1. CMake (cmake-gui)
- Where is the source code: C:/GitHub/llvm-project/
- Where to build the binaries: C:/GitHub/llvm-project/build
- Click the "Configure" button
  - Create Directory --> "Yes"
  - Specify the generator for this project: Visual Studio 17 2022
  - Optional platform for generator: x64
  - Optional toolset to use: ClangCL
  - Use default native compilers
  - Click the "Finish" button
- Modify the following entries
  - LLVM_ENABLE_PROJECTS: clang
  - LLVM_TARGETS_TO_BUILD: X86;AArch64
- Click the "Configure" button again
- Click the "Generate" button
- Click the "Open Project" button
2. Visual Studio is opened
- Change configuration from "Debug" to "Release"
- Build the "check-all" project


https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][lex] Fix non-portability diagnostics with absolute path (PR #74782)

2024-01-04 Thread Jeremy Wong via cfe-commits 

jrmwng wrote:

@jansvoboda11 the same error using the above steps

```
547>-- Build started: Project: check-all, Configuration: Release x64 --
547>Running all regression tests
547>llvm-lit.py: C:\GitHub\llvm-project\llvm\utils\lit\lit\llvm\config.py:46: 
note: using lit tools: C:\Program Files\Git\usr\bin
547>llvm-lit.py: C:\GitHub\llvm-project\llvm\utils\lit\lit\llvm\config.py:488: 
note: using clang: c:\github\llvm-project\build\release\bin\clang.exe
547>-- Testing: 70944 tests, 32 workers --
547>FAIL: Clang :: Lexer/case-insensitive-include-absolute.c (11482 of 70944)
547> TEST 'Clang :: 
Lexer/case-insensitive-include-absolute.c' FAILED 
547>Exit Code: 2
547>
547>Command Output (stdout):
547>--
547># RUN: at line 3
547>rm -rf 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp
 && split-file 
C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp
547># executed command: rm -rf 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp'
547># executed command: split-file 
'C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c' 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp'
547># RUN: at line 4
547>sed 
"s|DIR|C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp|g"
 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c.in
 > 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c
547># executed command: sed 
's|DIR|C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp|g'
 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c.in'
547># RUN: at line 5
547>c:\github\llvm-project\build\release\bin\clang.exe -cc1 -internal-isystem 
F:\GitHub\llvm-project\build\Release\lib\clang\18\include -nostdsysteminc 
-fsyntax-only 
C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c
 2>&1 | c:\github\llvm-project\build\release\bin\filecheck.exe 
C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c 
--DDIR=C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp
547># executed command: 'c:\github\llvm-project\build\release\bin\clang.exe' 
-cc1 -internal-isystem 
'F:\GitHub\llvm-project\build\Release\lib\clang\18\include' -nostdsysteminc 
-fsyntax-only 
'C:\GitHub\llvm-project\build\tools\clang\test\Lexer\Output\case-insensitive-include-absolute.c.tmp/tu.c'
547># executed command: 
'c:\github\llvm-project\build\release\bin\filecheck.exe' 
'C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c' 
--DDIR=C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp
547># .---command stderr
547>CUSTOMBUILD : # | FileCheck error : '' is empty.
547># | FileCheck command line:  
c:\github\llvm-project\build\release\bin\filecheck.exe 
C:\GitHub\llvm-project\clang\test\Lexer\case-insensitive-include-absolute.c 
--DDIR=C:/GitHub/llvm-project/build/tools/clang/test/Lexer/Output/case-insensitive-include-absolute.c.tmp
547># `-
547>CUSTOMBUILD : # error : command failed with exit status: 2
547>
547>--
547>
547>
547>
547>Failed Tests (1):
547>  Clang :: Lexer/case-insensitive-include-absolute.c
547>
547>
547>Testing Time: 433.52s
547>
547>Total Discovered Tests: 92696
547>  Skipped  :90 (0.10%)
547>  Unsupported  : 24219 (26.13%)
547>  Passed   : 68283 (73.66%)
547>  Expectedly Failed:   103 (0.11%)
547>  Failed   : 1 (0.00%)
547>C:\Program Files\Microsoft Visual 
Studio\2022\Community\MSBuild\Microsoft\VC\v170\Microsoft.CppCommon.targets(254,5):
 error MSB8066: Custom build for 
'C:\GitHub\llvm-project\build\CMakeFiles\5344a242bf0d5667ee3e0d3564612537\check-all.rule;C:\GitHub\llvm-project\llvm\CMakeLists.txt'
 exited with code 1.
547>Done building project "check-all.vcxproj" -- FAILED.
```

https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][lex] Fix non-portability diagnostics with absolute path (PR #74782)

2024-01-05 Thread Jeremy Wong via cfe-commits 

jrmwng wrote:

> @jrmwng PR #76985 (landed in 
> [853b133](https://github.com/llvm/llvm-project/commit/853b13342a131e06d61293ec6e840642054c6c85))
>  seems to have resolved this issue for me. Can you confirm that on your end?

Yes, your fix is effective 💯 

https://github.com/llvm/llvm-project/pull/74782
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits