[llvm] [Support] Remove an outdated MinGW workaround (PR #145294)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 23 02:01:45 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-platform-windows
Author: Martin Storsjö (mstorsjo)
<details>
<summary>Changes</summary>
mingw-w64 has had the _HEAPOK define since the initial commits in 2007; unclear when/where it was added for mingw.org headers, but it does seem to exist there as well (at least in versions from 2011).
This workaround stems from 53fbecce6e8b7d1f024e3dc6df4160fe9a577ff1 from 2004 - it is no longer relevant today.
---
Full diff: https://github.com/llvm/llvm-project/pull/145294.diff
1 Files Affected:
- (modified) llvm/lib/Support/Windows/Process.inc (-5)
``````````diff
diff --git a/llvm/lib/Support/Windows/Process.inc b/llvm/lib/Support/Windows/Process.inc
index d525f5b16e862..c4d18ca7375f2 100644
--- a/llvm/lib/Support/Windows/Process.inc
+++ b/llvm/lib/Support/Windows/Process.inc
@@ -37,11 +37,6 @@
//=== and must not be UNIX code
//===----------------------------------------------------------------------===//
-#ifdef __MINGW32__
-// This ban should be lifted when MinGW 1.0+ has defined this value.
-#define _HEAPOK (-2)
-#endif
-
using namespace llvm;
Process::Pid Process::getProcessId() {
``````````
</details>
https://github.com/llvm/llvm-project/pull/145294
More information about the llvm-commits
mailing list