cbiesinger created this revision.
cbiesinger added a reviewer: clayborg.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
In Python 3, PyInt doesn't exist (all integers are PyLongs). This patch
makes LLVM not use PyInt when compiling for Python 3. (Some code already
had such
cbiesinger abandoned this revision.
cbiesinger added a comment.
It seems this wasn't necessary, I had just misconfigured my build. (I don't
understand how this works though...)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64373/new/
https://revie
cbiesinger added a comment.
Ah, this works because Swig (?) adds compatibility macros to LLDBWrapPython:
#define PyInt_Check(x) PyLong_Check(x)
#define PyInt_AsLong(x) PyLong_AsLong(x)
#define PyInt_FromLong(x) PyLong_FromLong(x)
#define PyInt_FromSize_t(x) PyLong_FromSize_t(x)
(etc)
Repository:
cbiesinger created this revision.
cbiesinger added a reviewer: clayborg.
Herald added subscribers: lldb-commits, mgorny.
Herald added a project: LLDB.
Python 3.6 and 3.7 have been released.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D6
Files:
lldb/cmake/modules/LLDBSt
cbiesinger added a comment.
Thanks Greg! I'm not a committer -- how do I get this committed?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D6/new/
https://reviews.llvm.org/D6
___
lldb-commits ma
cbiesinger added a comment.
Yeah. (I thought I did? I'm not super familiar with this interface)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64373/new/
https://reviews.llvm.org/D64373
___
lldb-commits