[all-commits] [llvm/llvm-project] 2a6db9: [mlir][ods] Make OpBuilder and OperationState opti...

Jacques Pienaar via All-commits all-commits at lists.llvm.org
Tue Sep 22 10:04:50 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 2a6db92ca97da946307b559e63c6ac75caf4bbd6
      https://github.com/llvm/llvm-project/commit/2a6db92ca97da946307b559e63c6ac75caf4bbd6
  Author: Jacques Pienaar <jpienaar at google.com>
  Date:   2020-09-22 (Tue, 22 Sep 2020)

  Changed paths:
    M mlir/docs/OpDefinitions.md
    M mlir/test/lib/Dialect/Test/TestOps.td
    M mlir/test/mlir-tblgen/op-decl.td
    M mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp

  Log Message:
  -----------
  [mlir][ods] Make OpBuilder and OperationState optional

The OpBuilder is required to start with OpBuilder and OperationState, so remove
the need for the user to specify it. To make it simpler to update callers,
retain the legacy behavior for now and skip injecting OpBuilder/OperationState
when params start with OpBuilder.

Related to bug 47442.

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




More information about the All-commits mailing list