[all-commits] [llvm/llvm-project] ffe982: [lldb] Stop generating swig bindings for SBLaunchI...

Alex via All-commits all-commits at lists.llvm.org
Thu Feb 16 18:33:45 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ffe982f96bff77c1b326f5109e2dddfbcde4ca2a
      https://github.com/llvm/llvm-project/commit/ffe982f96bff77c1b326f5109e2dddfbcde4ca2a
  Author: Alex Langford <alangford at apple.com>
  Date:   2023-02-16 (Thu, 16 Feb 2023)

  Changed paths:
    M lldb/include/lldb/API/SBLaunchInfo.h

  Log Message:
  -----------
  [lldb] Stop generating swig bindings for SBLaunchInfo copy constructor

Given the line
```
launch_info = lldb.SBLaunchInfo(None)
```

We see different behaviors across different versionf of swig. On some
older versions of swig (e.g. 3.0.2) this line fails because it attempts
to use the copy constructor and blows up with an invalid null reference.
On newer versions of swig, this is correctly routed to the constructor
taking a pointer.

Prior to generating the swig bindings with the API headers,
SBLaunchInfo's copy constructor was not exposed so we're effectively
going back to the old behavior anyway.

Differential Revision: https://reviews.llvm.org/D144228




More information about the All-commits mailing list