This is an archive of the discontinued LLVM Phabricator instance.

[X86][Costmodel] `trunc v16i32 to v32i8/v64i8` can appear after legalization, cost is same as for `trunc v16i32 to v16i8`
ClosedPublic

Authored by lebedev.ri on Nov 14 2021, 10:08 AM.

Details

Summary

While this one is trivial and identical to the previous patch,
there is a weird cost change in a follow-up patch that i'm not sure about.

Diff Detail

Event Timeline

lebedev.ri created this revision.Nov 14 2021, 10:08 AM
RKSimon accepted this revision.Nov 15 2021, 7:34 AM

LGTM

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

LGTM

Thank you for the review!

This revision was landed with ongoing or failed builds.Nov 15 2021, 8:04 AM
This revision was automatically updated to reflect the committed changes.