This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] move from by-value to by-reference for data types
ClosedPublic

Authored by aartbik on Jun 16 2022, 5:12 PM.

Details

Summary

This fixes all sorts of ABI issues due to passing by-value
(using by-reference with memref's exclusively).

Diff Detail

Event Timeline

aartbik created this revision.Jun 16 2022, 5:12 PM
aartbik requested review of this revision.Jun 16 2022, 5:12 PM
aartbik updated this revision to Diff 437753.Jun 16 2022, 5:19 PM

fixed typo

aartbik updated this revision to Diff 437760.Jun 16 2022, 5:50 PM

added undef (missing in original code)

bkramer accepted this revision.Jun 17 2022, 12:09 AM

looks good

This revision is now accepted and ready to land.Jun 17 2022, 12:09 AM
This revision was landed with ongoing or failed builds.Jun 17 2022, 8:39 AM
This revision was automatically updated to reflect the committed changes.