Browse Source

Merge pull request #1602 from Faless/build/use_clang_cl_is_use_llvm

[SCons] Remove use_clang_cl windows flag in favor of generic use_llvm
David Snopek 10 months ago
parent
commit
6facde3c29
1 changed files with 2 additions and 3 deletions
  1. 2 3
      tools/windows.py

+ 2 - 3
tools/windows.py

@@ -76,10 +76,9 @@ def options(opts):
     mingw = os.getenv("MINGW_PREFIX", "")
     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_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("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("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)
     opts.Add("mingw_prefix", "MinGW prefix", mingw)
 
 
 
 
@@ -114,7 +113,7 @@ def generate(env):
         env.Append(CCFLAGS=["/utf-8"])
         env.Append(CCFLAGS=["/utf-8"])
         env.Append(LINKFLAGS=["/WX"])
         env.Append(LINKFLAGS=["/WX"])
 
 
-        if env["use_clang_cl"]:
+        if env["use_llvm"]:
             env["CC"] = "clang-cl"
             env["CC"] = "clang-cl"
             env["CXX"] = "clang-cl"
             env["CXX"] = "clang-cl"