This is an archive of the discontinued LLVM Phabricator instance.

[mlir][sparse] enable integral abs recognition
ClosedPublic

Authored by aartbik on Aug 11 2022, 12:28 PM.

Details

Summary

The end-to-end test for this new feature also exposed a bug
in LLVM IR lowering (since then, fixed), where we need to account
for the min-poison bit as extra argument.

declare i32 @llvm.abs.i32(i32 <src>, i1 <is_int_min_poison>)

Diff Detail

Event Timeline

aartbik created this revision.Aug 11 2022, 12:28 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2022, 12:28 PM
aartbik requested review of this revision.Aug 11 2022, 12:28 PM
bixia accepted this revision.Aug 11 2022, 3:59 PM
This revision is now accepted and ready to land.Aug 11 2022, 3:59 PM
aartbik edited the summary of this revision. (Show Details)Aug 12 2022, 9:53 AM
aartbik updated this revision to Diff 452241.Aug 12 2022, 11:10 AM

rebased with main, enabled the test for absi

This revision was landed with ongoing or failed builds.Aug 12 2022, 11:36 AM
This revision was automatically updated to reflect the committed changes.