This is an archive of the discontinued LLVM Phabricator instance.

[X86] Adding a test for vector shuffle extractions.
ClosedPublic

Authored by jbhateja on Aug 3 2017, 9:46 AM.

Details

Summary

When both the vector inputs of the shuffle vector is comprising of same
vector or shuffle mask is accessing elements from only one operand vector
(like in PR33758 test already present).

Diff Detail

Repository
rL LLVM

Event Timeline

jbhateja created this revision.Aug 3 2017, 9:46 AM
jbhateja accepted this revision.Aug 3 2017, 9:49 AM
jbhateja added reviewers: jbhateja, RKSimon.

Hi Simon,

Can you kindly land this revision into trunk. This will be updated with new results of D35788.

Thanks

This revision is now accepted and ready to land.Aug 3 2017, 9:49 AM
RKSimon edited edge metadata.Aug 3 2017, 9:53 AM

Hi Simon,

Can you kindly land this revision into trunk. This will be updated with new results of D35788.

Thanks

OK - but if you don't mind I'd like to put this in vector-shuffle-512-v64.ll instead of adding a new file?

Its ok, you can add it to already existing file.

I do not have commit rights so please merge this test.

Thanks

Hi Simon,

Can you kindly land this revision into trunk. This will be updated with new results of D35788.

Thanks

OK - but if you don't mind I'd like to put this in vector-shuffle-512-v64.ll instead of adding a new file?

Sorry, I should have said shuffle-vs-trunc-512.ll

You can add it where ever you think it shall make more sense.

Thanks

This revision was automatically updated to reflect the committed changes.