[PATCH] D73826: [mlir][Linalg][WIP][RFC] OpGen hooks

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 1 11:13:22 PST 2020


mehdi_amini added a comment.

In D73826#1853198 <https://reviews.llvm.org/D73826#1853198>, @jpienaar wrote:

> The previous rev was reverted as during discussion this feature was seen as unclear if we wanted to support, was changing the functionality & contract of the tool. Opening a new rev while rolling in the previous change does not resolve that discussion.


At least now there is a motivation / example of uses to support the discussion!

I'm still not convinced that hooking into Tablegen C++ is something we want Dialect author to do, so I'd like to look closer into what you're trying to do here and see how we could achieve this without injecting code into TableGen emitters.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73826





More information about the llvm-commits mailing list