This is an archive of the discontinued LLVM Phabricator instance.

[X86][SSE] Added more tests for vector multiplications as utility for D37896
ClosedPublic

Authored by pacxx on Sep 28 2017, 3:12 AM.

Details

Summary

Added additional tests for vector multiplications with multipliers that are:

  1. powers of 2 displaced by 1,
  2. product of a power of 2 displaced by one with another power of 2.

Diff Detail

Repository
rL LLVM

Event Timeline

pacxx created this revision.Sep 28 2017, 3:12 AM
RKSimon edited edge metadata.
RKSimon added a subscriber: llvm-commits.

What's going on here? It doesn't look like its been diffed against trunk.

Very minor, but please use [X86][SSE] instead of [Test] in your summary.

pacxx updated this revision to Diff 116982.Sep 28 2017, 7:08 AM
pacxx retitled this revision from [Test] Added more tests for vector multiplications as utility for D37896 to [X86][SSE] Added more tests for vector multiplications as utility for D37896.

Sorry somehow things messed up.

RKSimon accepted this revision.Sep 28 2017, 7:31 AM

LGTM - with one minor correction.

./test/CodeGen/X86/vector-mul.ll
1071 ↗(On Diff #116982)

v2i64 not v2i16

This revision is now accepted and ready to land.Sep 28 2017, 7:31 AM
pacxx updated this revision to Diff 116990.Sep 28 2017, 7:40 AM

fixed wrong type in function names

RKSimon accepted this revision.Sep 28 2017, 8:14 AM

LGTM, thanks

please commit the patch, so we can go on with D37896

This revision was automatically updated to reflect the committed changes.