Page MenuHomePhabricator

[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 Sun, Nov 14, 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.Sun, Nov 14, 10:08 AM
RKSimon accepted this revision.Mon, Nov 15, 7:34 AM

LGTM

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

LGTM

Thank you for the review!

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