This is an archive of the discontinued LLVM Phabricator instance.

[X86][CostModel] Improve costs for fp_to_uint/fp_to_sint for vXi8/vXi16/v2i32 results.
ClosedPublic

Authored by craig.topper on Apr 26 2020, 8:10 PM.

Details

Summary

This costs don't seem to have been updated very well after we
switched to widening legalization.

Diff Detail

Event Timeline

craig.topper created this revision.Apr 26 2020, 8:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 26 2020, 8:10 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
RKSimon accepted this revision.Apr 27 2020, 7:01 AM

LGTM - cheers

llvm/lib/Target/X86/X86TargetTransformInfo.cpp
1506

This probably means the AVX512 i32->i16/i8 truncation costs above need bumping as well

This revision is now accepted and ready to land.Apr 27 2020, 7:01 AM
This revision was automatically updated to reflect the committed changes.