[PATCH] D26082: Support for Python 3 in libclang python bindings

2017-01-11 Thread Mathieu Duponchelle via Phabricator via cfe-commits
MathieuDuponchelle added a comment. In https://reviews.llvm.org/D26082#621496, @jbcoe wrote: > Hi Mathieu, > What platform did you run the tests on? How did you run them? I don't get > any failures when running tests on macOS 10.12.1. Hey @jbcoe, do you think you can have a look at this issue

[PATCH] D26082: Support for Python 3 in libclang python bindings

2017-01-12 Thread Mathieu Duponchelle via Phabricator via cfe-commits
MathieuDuponchelle added a comment. In https://reviews.llvm.org/D26082#644402, @jbcoe wrote: > @MathieuDuponchelle I've set up a docker image with Fedora 23 and reproduced > the errors as you reported. > > I've no immediate idea what the problem might be. Might be worth looking with > address-s

[PATCH] D26082: Support for Python 3 in libclang python bindings

2017-01-13 Thread Mathieu Duponchelle via Phabricator via cfe-commits
MathieuDuponchelle added a comment. In https://reviews.llvm.org/D26082#645361, @jbcoe wrote: > I've reverted this change and submitted https://reviews.llvm.org/D28682 to > explicitly check the Python version. > > I'll pick this up again, I do want Python 3 support. I'm not entirely sure why yo

[PATCH] D26082: Support for Python 3 in libclang python bindings

2016-11-27 Thread Mathieu Duponchelle via Phabricator via cfe-commits
MathieuDuponchelle added a comment. Hey, for what it's worth when skipping the free at this line: https://github.com/llvm-mirror/clang/blob/master/bindings/python/clang/cindex.py#L181 the issues are gone, I haven't investigated the reason why. Repository: rL LLVM https://reviews.llvm.org/D2

[PATCH] D26082: Support for Python 3 in libclang python bindings

2016-12-14 Thread Mathieu Duponchelle via Phabricator via cfe-commits
MathieuDuponchelle added a comment. Hey, I'm on Linux, Fedora 23 to be specific. Repository: rL LLVM https://reviews.llvm.org/D26082 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit