[PATCH] D124269: [CostModel] Add basic fptoi_sat costs

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 04:14:46 PDT 2022


RKSimon added a comment.

You still need to get rid of the AVX checks - the script won't touch them once they aren't mentioned in any check-prefixes lists



================
Comment at: llvm/test/Analysis/CostModel/X86/fptoi_sat.ll:735
+;
+; AVX-LABEL: 'casts'
+; AVX-NEXT:  Cost Model: Found an estimated cost of 9 for instruction: %f32s1 = call i1 @llvm.fptosi.sat.i1.f32(float undef)
----------------
These need stripping now


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124269/new/

https://reviews.llvm.org/D124269



More information about the llvm-commits mailing list