This is an archive of the discontinued LLVM Phabricator instance.

[SLP][NFC] This adds a test for a follow-up patch that fixes a look-ahead operand reordering issue
ClosedPublic

Authored by vporpo on Mar 9 2022, 9:43 PM.

Diff Detail

Event Timeline

vporpo created this revision.Mar 9 2022, 9:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 9 2022, 9:43 PM
vporpo requested review of this revision.Mar 9 2022, 9:43 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 9 2022, 9:43 PM
RKSimon accepted this revision.Mar 9 2022, 11:33 PM

LGTM with one minor comment/question

llvm/test/Transforms/SLPVectorizer/X86/lookahead.ll
648

do we need the weak_odr attribute?

This revision is now accepted and ready to land.Mar 9 2022, 11:33 PM
vporpo updated this revision to Diff 414285.Mar 10 2022, 12:17 AM

Removed the weak_odr attribute.

RKSimon accepted this revision.Mar 10 2022, 12:42 AM
vporpo updated this revision to Diff 415943.Mar 16 2022, 12:51 PM

Added a new RUN for SSE2 checks.

This revision was landed with ongoing or failed builds.Mar 17 2022, 6:06 PM
This revision was automatically updated to reflect the committed changes.