This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][CostModel] Add test for fptoi_sat
Needs ReviewPublic

Authored by Miss_Grape on Aug 23 2022, 6:16 AM.

Details

Reviewers
reames

Diff Detail

Event Timeline

Miss_Grape created this revision.Aug 23 2022, 6:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 23 2022, 6:16 AM
Miss_Grape requested review of this revision.Aug 23 2022, 6:16 AM
Miss_Grape retitled this revision from [RISCV][CostModel] Add test forfptoi_sat to [RISCV][CostModel] Add test for fptoi_sat.

The costmodel setting of llvm.fptoui.sat is incorrect, need to do.

Miss_Grape abandoned this revision.Aug 24 2022, 2:16 AM

LGTM

Please rename add the scalable vector types as well.

You may find https://github.com/preames/riscv-llvm-cost-model-gen/ useful as a starting point for generating these.

Miss_Grape reclaimed this revision.Aug 25 2022, 6:23 PM
Miss_Grape removed a reviewer: dmgreen.