llvm-project
0773c83e - [lldb] Fix pid_t redefinition on Windows in ScriptInterpreterPythonInterfaces (#181493)

Commit
83 days ago
[lldb] Fix pid_t redefinition on Windows in ScriptInterpreterPythonInterfaces (#181493) ## Summary Include `lldb-python.h` as the first include inside the `LLDB_ENABLE_PYTHON` block in `ScriptInterpreterPythonInterfaces.cpp`, matching the pattern used by every other Python interface `.cpp` file in this directory. On Windows, `lldb-python.h` defines `NO_PID_T` before including `Python.h`. This prevents `PosixApi.h` (transitively included via `lldb-private.h`) from redefining `pid_t` with a conflicting type (`uint32_t` vs `int`). The issue was introduced by #181334 (ScriptedSymbolLocator plugin), which added a new header whose include chain transitively reaches `PosixApi.h`. Fixes Windows build failures on lldb-aarch64-windows, lldb-x86_64-win, and lldb-remote-linux-win. ## Test plan - [ ] lldb-aarch64-windows build passes - [ ] lldb-x86_64-win build passes - [ ] lldb-remote-linux-win build passes Co-authored-by: Rahul Reddy Chamala <rachamal@meta.com>
Author
Parents
Loading