iree-org / iree-llvm-sandbox

A sandbox for quick iteration and experimentation on projects related to IREE, MLIR, and LLVM
Apache License 2.0
54 stars 31 forks source link

[Substrait] Implement first version of NamedTableOp. #797

Closed ingomueller-net closed 7 months ago

ingomueller-net commented 8 months ago

This op corresponds to a Rel message with a ReadRel message that, in turn, has a NamedTable message. With this op, we are capable of representing a first complete plan in our dialect that we can serialize to protobuf and execute (by copy'n'pasting into an external consumer).

This PR is based on and, therefor, includes #795.

ingomueller-net commented 8 months ago

(just skimmed before next talk ... I should probably see how I could view stacked PRs better :) )

Thanks! In the "Files changed" tab, where you're probably doing the review, you can change "Show changes from all commits" to a single commit or even a range (by holding shift).