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 you reverted the patch altogether, apart from the 
problem I raised it worked flawlessly, and in any case did not create any 
problems when the bindings were used from python 2.


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-commits

Reply via email to