[PATCH] D86078: [AArch64] Improved lowering for saturating float to int.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 24 06:32:16 PDT 2020


paulwalker-arm added a comment.

Given you are making use of the fact that AArch64 fp_to_int already does the saturation part, I'm wondering if the different-width saturation can be moved from the input to the output?  Specially I'm thinking this will result in nicer code as the min/max immediate values will be easier to produce in integer form.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86078



More information about the llvm-commits mailing list