================
@@ -776,8 +796,13 @@ int main(int argc, char const *argv[]) {
                         "~/Library/Logs/DiagnosticReports/.\n");
 #endif
 
-#if defined(_WIN32) && defined(LLDB_PYTHON_DLL_RELATIVE_PATH)
-  AddPythonDLLToSearchPath();
+#if defined(_WIN32) && defined(LLDB_PYTHON_SHARED_LIBRARY_FILENAME)
+  if (!IsPythonDLLInPath())
+#ifdef LLDB_PYTHON_DLL_RELATIVE_PATH
----------------
mstorsjo wrote:

This condition nesting doesn't seem right. If we have 
`LLDB_PYTHON_DLL_RELATIVE_PATH` defined, but we don't have 
`LLDB_PYTHON_SHARED_LIBRARY_FILENAME`, then we no longer will call 
`AddPythonDLLToSearchPath` at all - which we did before. Not sure what the 
neatest form of this is; maybe an `#elif defined(LLDB_PYTHON_DLL_RELATIVE_PATH) 
AddPythonDLLToSearchPath(); #endif`?

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

Reply via email to