This is an archive of the discontinued LLVM Phabricator instance.

[X86][Costmodel] `trunc v16i32 to v32i16` can appear after legalization, cost is same as for `trunc v16i32 to v16i16`
ClosedPublic

Authored by lebedev.ri on Nov 14 2021, 5:58 AM.

Details

Summary

This was noticed in D113609, hopefully it unblocks that patch.
There are likely other similar problems.

Diff Detail

Event Timeline

lebedev.ri created this revision.Nov 14 2021, 5:58 AM
lebedev.ri requested review of this revision.Nov 14 2021, 5:58 AM
RKSimon accepted this revision.Nov 14 2021, 7:37 AM

LGTM

This revision is now accepted and ready to land.Nov 14 2021, 7:37 AM

LGTM

Thank you for the review!