Page MenuHomePhabricator

AggressiveInstCombine: Add tests for full multiplication pattern match
Needs ReviewPublic

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
Quuxplusone added inline comments.Jan 3 2019, 10:25 AM
test/Transforms/AggressiveInstCombine/mul_full_64.ll
63

Awesome! :)

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