[clang] Use the Windows SDK arguments over the environment (PR #144805)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 18 15:13:32 PDT 2025
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp -- clang/lib/Driver/ToolChains/MSVC.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/Driver/ToolChains/MSVC.cpp b/clang/lib/Driver/ToolChains/MSVC.cpp
index 632cc3a0a..32a237323 100644
--- a/clang/lib/Driver/ToolChains/MSVC.cpp
+++ b/clang/lib/Driver/ToolChains/MSVC.cpp
@@ -96,11 +96,10 @@ void visualstudio::Linker::ConstructJob(Compilation &C, const JobAction &JA,
// the environment variable is set however, assume the user knows what
// they're doing. If the user passes /vctoolsdir or /winsdkdir or any of the
// other Windows SDK options, trust that over env vars.
- const Arg *A = Args.getLastArg(options::OPT__SLASH_vctoolsdir,
- options::OPT__SLASH_vctoolsversion,
- options::OPT__SLASH_winsysroot,
- options::OPT__SLASH_winsdkdir,
- options::OPT__SLASH_winsdkversion);
+ const Arg *A = Args.getLastArg(
+ options::OPT__SLASH_vctoolsdir, options::OPT__SLASH_vctoolsversion,
+ options::OPT__SLASH_winsysroot, options::OPT__SLASH_winsdkdir,
+ options::OPT__SLASH_winsdkversion);
if (A) {
// cl.exe doesn't find the DIA SDK automatically, so this too requires
// explicit flags and doesn't automatically look in "DIA SDK" relative
@@ -681,11 +680,10 @@ void MSVCToolChain::AddClangSystemIncludeArgs(const ArgList &DriverArgs,
// Windows SDK options. If so, we assume the user knows what they're doing
// and don't try to find the include directories automatically.
// If not, we try to find the include directories automatically.
- const Arg *A = DriverArgs.getLastArg(options::OPT__SLASH_vctoolsdir,
- options::OPT__SLASH_vctoolsversion,
- options::OPT__SLASH_winsysroot,
- options::OPT__SLASH_winsdkdir,
- options::OPT__SLASH_winsdkversion);
+ const Arg *A = DriverArgs.getLastArg(
+ options::OPT__SLASH_vctoolsdir, options::OPT__SLASH_vctoolsversion,
+ options::OPT__SLASH_winsysroot, options::OPT__SLASH_winsdkdir,
+ options::OPT__SLASH_winsdkversion);
// Add DIA SDK include if requested.
if (A) {
``````````
</details>
https://github.com/llvm/llvm-project/pull/144805
More information about the cfe-commits
mailing list