This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by lebedev.ri on Nov 14 2021, 6:19 AM.

Diff Detail

Event Timeline

lebedev.ri created this revision.Nov 14 2021, 6:19 AM
RKSimon accepted this revision.Nov 14 2021, 6:43 AM

LGTM

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

LGTM

Thank you for the review. I assume the i32 version (D113842) is also fine?

This revision was landed with ongoing or failed builds.Nov 14 2021, 7:42 AM
This revision was automatically updated to reflect the committed changes.