[PATCH] D102353: [AArch64] Lower fpto*i.sat intrinsics.

[Jacob Bramley via Phabricator via llvm-commits]

jbramley marked 10 inline comments as done.
jbramley added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelLowering.cpp:16032
   case ISD::FP_TO_UINT:
+    // TODO: Do the same for FP_TO_*INT_SAT.
     return performFpToIntCombine(N, DAG, DCI, Subtarget);
----------------
SjoerdMeijer wrote:
> Nit: the comment makes sense, I am not sure it belongs here though.
Agreed. It probably makes more sense near the `setTargetDAGCombine` configuration in `AArch64TargetLowering`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102353