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

[David Sherwood via Phabricator via llvm-commits]

david-arm added a comment.

Hi @dmgreen, just out of curiosity why do we only care about NaNs for the signed case? Looking at the documentation for llvm.fptosi.sat and llvm.fptoui.sat they both state that "if any argument is NaN, zero is returned". So don't you have to do a fcmp+select in both cases?


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

https://reviews.llvm.org/D124269