[PATCH] D99935: [AArch64] Add instruction costs for FP_TO_UINT and FP_TO_SINT with half types

[David Sherwood via Phabricator via llvm-commits]

david-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:533
+    // Complex, from nxv8f64.
+    { ISD::FP_TO_SINT, MVT::nxv8i32, MVT::nxv8f64, 6 },
+    { ISD::FP_TO_SINT, MVT::nxv8i16, MVT::nxv8f64, 7 },
----------------
After private discussion with @sdesmalen it turns out we shouldn't need entries for illegal -> illegal types as they will both be split in the BasicTTIImpl::getCastInstrCost function.


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

https://reviews.llvm.org/D99935