This is an archive of the discontinued LLVM Phabricator instance.

[mlir][spirv] Fix SPIR-V translation registration
ClosedPublic

Authored by antiagainst on Apr 27 2020, 8:36 AM.

Details

Summary

Now we must explicitly call the registration function given that
MLIR moved away from static registration.

Diff Detail

Event Timeline

antiagainst created this revision.Apr 27 2020, 8:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 27 2020, 8:36 AM
mravishankar accepted this revision.Apr 27 2020, 9:39 AM

Just a nit about commit message. This should also expose -deserialize-spirv right?

This revision is now accepted and ready to land.Apr 27 2020, 9:39 AM

Address comments

Just a nit about commit message. This should also expose -deserialize-spirv right?

It's actually -serialize-spirv. But yes indeed the registration function name should be swapped. Fixed.

antiagainst retitled this revision from [mlir][spirv] Expose -serialize-spirv via mlir-translate to [mlir][spirv] Fix SPIR-V translation registration.Apr 27 2020, 10:15 AM
This revision was automatically updated to reflect the committed changes.