[all-commits] [llvm/llvm-project] 9b2288: [mlir][ODS] Switch declarative rewrite rules to pr...

[Krzysztof Drewniak via All-commits]

  Branch: refs/heads/users/krzysz00/declaritve-rewrite-with-properties
  Home:   https://github.com/llvm/llvm-project
  Commit: 9b2288bde8bb68e6900a5c41d2b5162c6e50fae8
      https://github.com/llvm/llvm-project/commit/9b2288bde8bb68e6900a5c41d2b5162c6e50fae8
  Author: Krzysztof Drewniak <krzysdrewniak at gmail.com>
  Date:   2025-03-10 (Mon, 10 Mar 2025)

  Changed paths:
    A mlir/test/mlir-tblgen/rewriter-attributes-properties.td
    M mlir/tools/mlir-tblgen/RewriterGen.cpp

  Log Message:
  -----------
  [mlir][ODS] Switch declarative rewrite rules to properties structs

Now that we have collective builders that take
`const [RelevantOp]::Properties &` arguments, we don't need to serialize
all the attributes that'll be set during an output pattern into a dictionary
attribute. Similarly, we can use the properties struct to get the attributes
instead of needing to go through the big if statement in getAttrOfType<>().

This also enables us to have declarative rewrite rules that match non-attribute
properties in a future PR.

This commit also adds a basic test for the generated matchers since there
didn't seem to already be one.


  Commit: dbd8b51f0d27e3f2ba035c0dc2b27a95cdfea806
      https://github.com/llvm/llvm-project/commit/dbd8b51f0d27e3f2ba035c0dc2b27a95cdfea806
  Author: Krzysztof Drewniak <krzysdrewniak at gmail.com>
  Date:   2025-03-10 (Mon, 10 Mar 2025)

  Changed paths:
    M mlir/test/mlir-tblgen/rewriter-attributes-properties.td

  Log Message:
  -----------
  Test fails on Windows, try to loosen it up


  Commit: 0ce3cecc51d79cc187a3afcd3cdccc4b7cff903b
      https://github.com/llvm/llvm-project/commit/0ce3cecc51d79cc187a3afcd3cdccc4b7cff903b
  Author: Krzysztof Drewniak <krzysdrewniak at gmail.com>
  Date:   2025-03-10 (Mon, 10 Mar 2025)

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

  Log Message:
  -----------
  Review suggestions


Compare: https://github.com/llvm/llvm-project/compare/3abe8424488d...0ce3cecc51d7

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