[all-commits] [llvm/llvm-project] d65936: [Support][Cygwin] Fix handling of Process symbol l...

[jeremyd2019 via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d659364295f6f0c41535a0c98c958cb4c896cc62
      https://github.com/llvm/llvm-project/commit/d659364295f6f0c41535a0c98c958cb4c896cc62
  Author: jeremyd2019 <github at jdrake.com>
  Date:   2025-06-09 (Mon, 09 Jun 2025)

  Changed paths:
    M llvm/lib/Support/DynamicLibrary.cpp
    M llvm/lib/Support/Unix/DynamicLibrary.inc
    M llvm/lib/Support/Windows/DynamicLibrary.inc

  Log Message:
  -----------
  [Support][Cygwin] Fix handling of Process symbol lookup. (#143072)

In Unix/DynamicLibrary.inc, it was already known that Cygwin required
use of `RTLD_DEFAULT` as the `Handle` parameter to `DLSym` to search all
modules for a symbol. Unfortunately, RTLD_DEFAULT is defined as NULL, so
the existing checks of the `Process` handle meant `DLSym` would never be
called on Cygwin. Use the existing `&Invalid` sentinel instead of
`nullptr` for the `Process` handle.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications