Page MenuHomePhabricator

AggressiveInstCombine: Add tests for full multiplication pattern match
Needs ReviewPublic

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



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

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

Awesome! :)

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