This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] minor cleanup of include placement
ClosedPublic

Authored by aartbik on Feb 11 2022, 9:25 PM.

Details

Summary

Rationale:
empty line between main include for this file
moved include that actually defines code into right section

Note that this revision started as breaking up ops/attrs even more
(for bug https://github.com/llvm/llvm-project/issues/52748), but due
the the connection in Dialect.initalize(), this cannot be split further).
All heavy lifting refactoring was already done by River in previous cleanup.

Diff Detail

Event Timeline

aartbik created this revision.Feb 11 2022, 9:25 PM
aartbik requested review of this revision.Feb 11 2022, 9:25 PM
bixia accepted this revision.Feb 13 2022, 2:53 PM
This revision is now accepted and ready to land.Feb 13 2022, 2:53 PM
This revision was automatically updated to reflect the committed changes.