Is there a particular reason why this and https://reviews.llvm.org/D55259 are separate patches?
The Python 2/3 portability is a huge patch. I have split it in several micro-patch, one for each package/feature impacted. The underlying idea is that we can keep the discussions focus. This package should just be a renaming issue, while https://reviews.llvm.org/D55259 uses a higher level API than the one before.
LGTM with one minor nit.
|90 ↗||(On Diff #176572)|