llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang-driver

Author: Matt Arsenault (arsenm)

<details>
<summary>Changes</summary>

The comment says it's needed for getenv, but this file does
not directly use getenv; it uses sys::Process::GetEnv.

---
Full diff: https://github.com/llvm/llvm-project/pull/209707.diff


1 Files Affected:

- (modified) clang/lib/Driver/Driver.cpp (-1) 


``````````diff
diff --git a/clang/lib/Driver/Driver.cpp b/clang/lib/Driver/Driver.cpp
index e606cdc4c1cf8..a7fcc30291e09 100644
--- a/clang/lib/Driver/Driver.cpp
+++ b/clang/lib/Driver/Driver.cpp
@@ -110,7 +110,6 @@
 #include "llvm/Support/raw_ostream.h"
 #include "llvm/TargetParser/Host.h"
 #include "llvm/TargetParser/RISCVISAInfo.h"
-#include <cstdlib> // ::getenv
 #include <map>
 #include <memory>
 #include <optional>

``````````

</details>


https://github.com/llvm/llvm-project/pull/209707
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to