This is an archive of the discontinued LLVM Phabricator instance.

[X86][SSE] Added general integer shuffle matching for MOVQ instruction
ClosedPublic

Authored by RKSimon on Jan 29 2015, 8:48 AM.

Details

Summary

This patch adds general shuffle pattern matching for the MOVQ zero-extend instruction (copy lower 64bits, zero upper) for all 128-bit integer vectors, it is added as a fallback test in lowerVectorShuffleAsZeroOrAnyExtend.

Regarding the issues in the tests - combine-or.ll is fixed in conjunction with D6649 (which reduces it to a shift + movq) and the superfluous movq after the pshufb in vector-shuffle-128-v16.ll are resolved once x86-experimental-vector-shuffle-legality is enabled.

This is a follow-up to D7228 which adds shuffle decodes / assembly comments for the movq comments - if that ticket gets approved I can update this patch's tests to include the comments.

Note I have other patterns to add to lowerVectorShuffleAsZeroOrAnyExtend shortly.

Diff Detail

Repository
rL LLVM

Event Timeline

RKSimon updated this revision to Diff 18967.Jan 29 2015, 8:48 AM
RKSimon retitled this revision from to [X86][SSE] Added general integer shuffle matching for MOVQ instruction.
RKSimon updated this object.
RKSimon edited the test plan for this revision. (Show Details)
RKSimon set the repository for this revision to rL LLVM.
RKSimon added a subscriber: Unknown Object (MLST).
RKSimon updated this revision to Diff 19092.Jan 31 2015, 8:17 AM

Updated with assembly comments now that D7228 has been committed.

Also added additional tests against 4i32 and 8i16 vectors.

chandlerc accepted this revision.Feb 3 2015, 10:20 AM
chandlerc edited edge metadata.

Minor comments. Good to go with these.

lib/Target/X86/X86ISelLowering.cpp
7989

I would call this "CanZExtLowHalf", and document that it returns the input whose low half can be zext-ed.

Also, no need to use an explicit return in the lambda, it should be deduced.

8001

I would call the variable 'V'. It isn't a MOVQ (yet).

This revision is now accepted and ready to land.Feb 3 2015, 10:20 AM
RKSimon closed this revision.Feb 3 2015, 12:14 PM

http://reviews.llvm.org/rL228022

Thanks Chandler, committed with your changes.