[all-commits] [llvm/llvm-project] 5b7c91: [mlir][ODS] Add a collective builder that takes th...

[Krzysztof Drewniak via All-commits]

  Branch: refs/heads/users/krzysz00/collective-prop-builder
  Home:   https://github.com/llvm/llvm-project
  Commit: 5b7c91328d935f203acd571488f16f709200cc1e
      https://github.com/llvm/llvm-project/commit/5b7c91328d935f203acd571488f16f709200cc1e
  Author: Krzysztof Drewniak <krzysdrewniak at gmail.com>
  Date:   2025-02-05 (Wed, 05 Feb 2025)

  Changed paths:
    M mlir/docs/DeclarativeRewrites.md
    M mlir/docs/DefiningDialects/Operations.md
    M mlir/include/mlir/IR/OpDefinition.h
    M mlir/include/mlir/IR/OperationSupport.h
    M mlir/test/lib/Dialect/Test/TestOps.td
    M mlir/test/mlir-tblgen/op-attribute.td
    M mlir/test/mlir-tblgen/op-decl-and-defs.td
    M mlir/test/mlir-tblgen/op-result.td
    M mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp

  Log Message:
  -----------
  [mlir][ODS] Add a collective builder that takes the Properties struct

This commit adds builders of the form

```
static void build(..., [TypeRange resultTypes],
                  ValueRange operands, const Properties &properties,
                  ArrayRef<NamedAttribute> discardableAttributes = {},
                  [unsigned numRegions]);
```
to go alongside the existing
result/operands/[inherent + discardable attribute list] collective
builders.

This change is intended to support a refactor to the declarative
rewrite engine to make it populate the `Properties` struct instead of
creating a `DictionaryAttr`, thus enabling rewrite rules to handle
non-`Attribute` properties.

More generally, this means that generic code that would previously
call `getAttrs()` to blend together inherent and discardable
attributes can now use `getProperties()` and `getDiscardableAttrs()`
separately, thus removing the need to serialize everything into a
temporary `DictionaryAttr`.


  Commit: 8e5416c856946d0fa1be8278f89ce9e92c877ee2
      https://github.com/llvm/llvm-project/commit/8e5416c856946d0fa1be8278f89ce9e92c877ee2
  Author: Krzysztof Drewniak <krzysdrewniak at gmail.com>
  Date:   2025-02-05 (Wed, 05 Feb 2025)

  Changed paths:
    M mlir/tools/mlir-tblgen/OpDefinitionsGen.cpp

  Log Message:
  -----------
  Clang-format


  Commit: a65fe2d0394346cab1751e49891a3012eadfca6e
      https://github.com/llvm/llvm-project/commit/a65fe2d0394346cab1751e49891a3012eadfca6e
  Author: Krzysztof Drewniak <krzysdrewniak at gmail.com>
  Date:   2025-02-05 (Wed, 05 Feb 2025)

  Changed paths:
    M mlir/unittests/TableGen/OpBuildGen.cpp

  Log Message:
  -----------
  Right, there was supposed to be a unit test here


Compare: https://github.com/llvm/llvm-project/compare/97bdf6d7970f...a65fe2d03943

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications