Skip to content

Commit c7d44ba

Browse files
StefanStojanovictargos
authored andcommitted
build: find version of Clang installed on Windows
PR-URL: #53228 Refs: #52809 Reviewed-By: Michaël Zasso <targos@protonmail.com>
1 parent 24c7a94 commit c7d44ba

File tree

3 files changed

+37
-5
lines changed

3 files changed

+37
-5
lines changed

BUILDING.md

+9-1
Original file line numberDiff line numberDiff line change
@@ -642,14 +642,22 @@ Optional requirements to build the MSI installer package:
642642
* The .NET SDK component from [Visual Studio 2022](https://visualstudio.microsoft.com/downloads/)
643643
* This component can be installed via the Visual Studio Installer Application
644644

645-
Optional requirements for compiling for Windows 10 on ARM (ARM64):
645+
Optional requirements for compiling for Windows on ARM (ARM64):
646646

647647
* Visual Studio 17.6.0 or newer
648648
* Visual Studio optional components
649649
* Visual C++ compilers and libraries for ARM64
650650
* Visual C++ ATL for ARM64
651651
* Windows 10 SDK 10.0.17763.0 or newer
652652

653+
Optional requirements for compiling with ClangCL:
654+
655+
* Visual Studio optional components
656+
* C++ Clang Compiler for Windows
657+
* MSBuild support for LLVM toolset
658+
659+
NOTE: Currently we only support compiling with Clang that comes from Visual Studio.
660+
653661
##### Option 2: Automated install with Boxstarter
654662

655663
A [Boxstarter](https://boxstarter.org/) script can be used for easy setup of

tools/msvs/vswhere_usability_wrapper.cmd

+5
Original file line numberDiff line numberDiff line change
@@ -13,11 +13,16 @@ if not exist "%InstallerPath%" goto :no-vswhere
1313
set "Path=%Path%;%InstallerPath%"
1414
where vswhere 2> nul > nul
1515
if errorlevel 1 goto :no-vswhere
16+
:: VC.Tools are needed even when using clang because of the vcvarsall.bat usage
1617
if "%2"=="arm64" (
1718
set VSWHERE_REQ=-requires Microsoft.VisualStudio.Component.VC.Tools.ARM64
1819
) else (
1920
set VSWHERE_REQ=-requires Microsoft.VisualStudio.Component.VC.Tools.x86.x64
2021
)
22+
if "%4"=="1" (
23+
set VSWHERE_REQ=%VSWHERE_REQ% -requires Microsoft.VisualStudio.Component.VC.Llvm.Clang
24+
set VSWHERE_REQ=%VSWHERE_REQ% -requires Microsoft.VisualStudio.Component.VC.Llvm.ClangToolset
25+
)
2126
set VSWHERE_PRP=-property installationPath
2227
set VSWHERE_LMT=-version %1
2328
vswhere -prerelease > nul

vcbuild.bat

+23-4
Original file line numberDiff line numberDiff line change
@@ -187,8 +187,6 @@ if defined nosnapshot set configure_flags=%configure_flags% --without-snap
187187
if defined nonpm set configure_flags=%configure_flags% --without-npm
188188
if defined nocorepack set configure_flags=%configure_flags% --without-corepack
189189
if defined ltcg set configure_flags=%configure_flags% --with-ltcg
190-
:: If clang-cl build is requested, set it to 17.0, which is the version shipped with VS 2022.
191-
if defined clang_cl set configure_flags=%configure_flags% --clang-cl=17.0
192190
if defined release_urlbase set configure_flags=%configure_flags% --release-urlbase=%release_urlbase%
193191
if defined download_arg set configure_flags=%configure_flags% %download_arg%
194192
if defined enable_vtune_arg set configure_flags=%configure_flags% --enable-vtune-profiling
@@ -254,7 +252,7 @@ echo Looking for Visual Studio 2022
254252
@rem cleared first as vswhere_usability_wrapper.cmd doesn't when it fails to
255253
@rem detect the version searched for
256254
if not defined target_env set "VCINSTALLDIR="
257-
call tools\msvs\vswhere_usability_wrapper.cmd "[17.6,18.0)" %target_arch% "prerelease"
255+
call tools\msvs\vswhere_usability_wrapper.cmd "[17.6,18.0)" %target_arch% "prerelease" %clang_cl%
258256
if "_%VCINSTALLDIR%_" == "__" goto msbuild-not-found
259257
@rem check if VS2022 is already setup, and for the requested arch
260258
if "_%VisualStudioVersion%_" == "_17.0_" if "_%VSCMD_ARG_TGT_ARCH%_"=="_%target_arch%_" goto found_vs2022
@@ -274,13 +272,34 @@ set PLATFORM_TOOLSET=v143
274272
goto msbuild-found
275273

276274
:msbuild-not-found
277-
echo Failed to find a suitable Visual Studio installation.
275+
set "clang_echo="
276+
if defined clang_cl set "clang_echo= or Clang compiler/LLVM toolset"
277+
echo Failed to find a suitable Visual Studio installation%clang_echo%.
278278
echo Try to run in a "Developer Command Prompt" or consult
279279
echo https://github.com/nodejs/node/blob/HEAD/BUILDING.md#windows
280280
goto exit
281281

282282
:msbuild-found
283283

284+
@rem check if the clang-cl build is requested
285+
if not defined clang_cl goto clang-skip
286+
@rem x64 is hard coded as it is used for both cross and native compilation.
287+
set "clang_path=%VCINSTALLDIR%\Tools\Llvm\x64\bin\clang.exe"
288+
for /F "tokens=3" %%i in ('"%clang_path%" --version') do (
289+
set clang_version=%%i
290+
goto clang-found
291+
)
292+
293+
:clang-not-found
294+
echo Failed to find Clang compiler in %clang_path%.
295+
goto exit
296+
297+
:clang-found
298+
echo Found Clang version %clang_version%
299+
set configure_flags=%configure_flags% --clang-cl=%clang_version%
300+
301+
:clang-skip
302+
284303
set project_generated=
285304
:project-gen
286305
@rem Skip project generation if requested.

0 commit comments

Comments
 (0)