This is an archive of the discontinued LLVM Phabricator instance.

[mlir] Move LLVM Dialect Op documentation to ODS
ClosedPublic

Authored by ftynse on Dec 16 2020, 3:55 PM.

Details

Summary

This was long overdue. The initial documentation for the LLVM dialect was
introduced before ODS had support for long descriptions. This is now possible,
so the documentation is moved to ODS, which can serve as a single source of
truth. The high-level description of the dialect structure is updated to
reflect that.

Depends On: D93315

Diff Detail

Event Timeline

ftynse created this revision.Dec 16 2020, 3:55 PM
ftynse requested review of this revision.Dec 16 2020, 3:55 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 16 2020, 3:55 PM
mehdi_amini accepted this revision.Dec 16 2020, 7:58 PM
mehdi_amini added inline comments.
mlir/docs/Dialects/LLVM.md
59
This revision is now accepted and ready to land.Dec 16 2020, 7:58 PM
rriddle accepted this revision.Dec 16 2020, 7:59 PM

Nice!

mlir/include/mlir/Dialect/LLVMIR/LLVMOps.td
412

Can you indent these descriptions? It should still be formatted properly when generated.

ftynse updated this revision to Diff 312425.Dec 17 2020, 3:21 AM
ftynse marked 2 inline comments as done.

Address review.

This revision was landed with ongoing or failed builds.Dec 17 2020, 3:33 AM
This revision was automatically updated to reflect the committed changes.