This is an archive of the discontinued LLVM Phabricator instance.

AggressiveInstCombine: Add tests for full multiplication pattern match
AbandonedPublic

Authored by chfast on Jan 3 2019, 9:42 AM.

Details

Summary

The AggressiveInstCombine pass does not support this optimization yet. The tests are added to observe the result of implementing the full multiplication pattern match.

These tests are added as requested in https://reviews.llvm.org/D56214.

Diff Detail

Event Timeline

chfast created this revision.Jan 3 2019, 9:42 AM
chfast edited the summary of this revision. (Show Details)Jan 3 2019, 9:47 AM
test/Transforms/AggressiveInstCombine/mul_full_64.ll
63

Awesome! :)

Right. The existing tests seem ok.
Probably some negative tests will be needed though.

lebedev.ri resigned from this revision.Jun 12 2020, 5:28 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 12 2020, 5:28 AM
chfast abandoned this revision.Nov 2 2022, 5:17 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 2 2022, 5:17 AM
Herald added a subscriber: StephenFan. · View Herald Transcript