[all-commits] [llvm/llvm-project] b4c93e: [MLIR] [Python ODS] Use @builtins.property for cas...

John Demme via All-commits all-commits at lists.llvm.org
Thu Jul 15 19:39:54 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b4c93ece8e4f6e98a15daca10c8a3db33cf8c195
      https://github.com/llvm/llvm-project/commit/b4c93ece8e4f6e98a15daca10c8a3db33cf8c195
  Author: John Demme <john.demme at microsoft.com>
  Date:   2021-07-15 (Thu, 15 Jul 2021)

  Changed paths:
    M mlir/test/mlir-tblgen/op-python-bindings.td
    M mlir/test/python/python_test_ops.td
    M mlir/tools/mlir-tblgen/OpPythonBindingGen.cpp

  Log Message:
  -----------
  [MLIR] [Python ODS] Use @builtins.property for cases where 'property' is already defined

In cases where an operation has an argument or result named 'property', the
ODS-generated python fails on import because the `@property` resolves to the
`property` operation argument instead of the builtin `@property` decorator. We
should always use the fully qualified decorator name.

Reviewed By: mikeurbach

Differential Revision: https://reviews.llvm.org/D106106




More information about the All-commits mailing list