[PATCH] D67879: [LNT] Python 3 support: import object when inheriting from it

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 7 07:55:39 PDT 2019


hubert.reinterpretcast added a comment.

My understanding is that this patch has no effect for Python 3. In Python 2, `object` from `builtins` (as provided by the `future` package) is used to enable use of some Python 3 coding patterns. Absent further changes that make use of such enablement, I am not sure that this patch is necessary. If this patch is needed to support later patches, then I suggest applying the fixer at that point. Applying the fixer before applying D67882 <https://reviews.llvm.org/D67882> seems odd anyway.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67879/new/

https://reviews.llvm.org/D67879





More information about the llvm-commits mailing list