This is an archive of the discontinued LLVM Phabricator instance.

[COST]Fix PR35865: Fix cost model evaluation for shuffle on X86.
ClosedPublic

Authored by ABataev on Jan 9 2018, 7:07 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ABataev created this revision.Jan 9 2018, 7:07 AM
RKSimon accepted this revision.Jan 9 2018, 9:43 AM

LGTM with one minor

test/Transforms/SLPVectorizer/X86/crash_shuffle.ll
2 ↗(On Diff #129077)

Rename this pr35865.ll ? Its useful not to have to goto svn log to find where a test came from (and crash_shuffle.ll isn't that useful a name....).

This revision is now accepted and ready to land.Jan 9 2018, 9:43 AM
This revision was automatically updated to reflect the committed changes.