Skip to content

Commit

Permalink
Merge pull request #1602 from Faless/build/use_clang_cl_is_use_llvm
Browse files Browse the repository at this point in the history
[SCons] Remove use_clang_cl windows flag in favor of generic use_llvm
  • Loading branch information
dsnopek authored Oct 3, 2024
2 parents 709bad0 + 4717a78 commit 6facde3
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions tools/windows.py
Original file line number Diff line number Diff line change
Expand Up @@ -76,10 +76,9 @@ def options(opts):
mingw = os.getenv("MINGW_PREFIX", "")

opts.Add(BoolVariable("use_mingw", "Use the MinGW compiler instead of MSVC - only effective on Windows", False))
opts.Add(BoolVariable("use_clang_cl", "Use the clang driver instead of MSVC - only effective on Windows", False))
opts.Add(BoolVariable("use_static_cpp", "Link MinGW/MSVC C++ runtime libraries statically", True))
opts.Add(BoolVariable("silence_msvc", "Silence MSVC's cl/link stdout bloat, redirecting errors to stderr.", True))
opts.Add(BoolVariable("use_llvm", "Use the LLVM compiler", False))
opts.Add(BoolVariable("use_llvm", "Use the LLVM compiler (MVSC or MinGW depending on the use_mingw flag)", False))
opts.Add("mingw_prefix", "MinGW prefix", mingw)


Expand Down Expand Up @@ -114,7 +113,7 @@ def generate(env):
env.Append(CCFLAGS=["/utf-8"])
env.Append(LINKFLAGS=["/WX"])

if env["use_clang_cl"]:
if env["use_llvm"]:
env["CC"] = "clang-cl"
env["CXX"] = "clang-cl"

Expand Down

0 comments on commit 6facde3

Please sign in to comment.