This is an archive of the discontinued LLVM Phabricator instance.

WIP: Test case that exposes an SLP vectorization missed opportunity
AbandonedPublic

Authored by rsanthir.quic on Nov 12 2020, 4:09 PM.

Details

Reviewers
apazos
fhahn
Summary

I have tried to produce a reduced testcase that shows the operand reordering issue, and was only able to reduce it to a 8-wide vectorization. In the added test when the following IR change is made, vectorization successfully occurs. The question is can SLP be made to recognize this and proceed on its own.

Change:
%add = add nuw nsw i32 %conv10, %conv7

To:
%add = add nuw nsw i32 %conv7, %conv10

Diff Detail

Event Timeline

rsanthir.quic created this revision.Nov 12 2020, 4:09 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 12 2020, 4:09 PM
rsanthir.quic requested review of this revision.Nov 12 2020, 4:09 PM
rsanthir.quic edited the summary of this revision. (Show Details)
rsanthir.quic abandoned this revision.Nov 12 2020, 4:19 PM

Incomplete patch, accidentally pushed