This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Add TFFramework dialect to DialectSymbolRegistry.
ClosedPublic

Authored by pifon2a on Jul 30 2020, 1:47 AM.

Details

Summary

TF Framework is a "host side" TF dialect.

Diff Detail

Event Timeline

pifon2a created this revision.Jul 30 2020, 1:47 AM
Herald added a project: Restricted Project. · View Herald Transcript
pifon2a requested review of this revision.Jul 30 2020, 1:47 AM

Looks reasonable to me. However, we are registering this for a dialect that is in very local use. Maybe @rriddle can advise.

mehdi_amini accepted this revision.Jul 30 2020, 12:07 PM

If you are just experimenting with it, you can use one of the PRIVATE_EXPERIMENTAL one, otherwise this is intended to be "cheap" to add.

This revision is now accepted and ready to land.Jul 30 2020, 12:07 PM
This revision was automatically updated to reflect the committed changes.

If you are just experimenting with it, you can use one of the PRIVATE_EXPERIMENTAL one, otherwise this is intended to be "cheap" to add.

Thank you, Mehdi. We've been using PRIVATE_EXPERIMENTAL_0 for a while already.