Page MenuHomePhabricator

[X86] Fix pshuflw formation from repeated shuffle mask (PR43230)

Authored by nikic on Sep 7 2019, 3:51 AM.



Fix for

When creating PSHUFLW from a repeated shuffle mask, we have to apply the checks to the repeated mask, not the original one. For the test case from PR43230 the inspected part of the original mask is all undef.

Diff Detail


Event Timeline

nikic created this revision.Sep 7 2019, 3:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 7 2019, 3:51 AM
lebedev.ri added inline comments.
4757 ↗(On Diff #219229)

Precommit, and note that 14'th channel shuffle is out-of-bounds

RKSimon accepted this revision.Sep 7 2019, 4:58 AM

Nice catch! LGTM, as @lebedev.ri suggested please pre-commit the test so when the fix is committed it shows the codegen diff.

This revision is now accepted and ready to land.Sep 7 2019, 4:58 AM
This revision was automatically updated to reflect the committed changes.