[PATCH] D98934: [SVE] Add instruction cost for fptrunc in loops

[Nashe Mncube via Phabricator via llvm-commits]

nasherm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:461
+      // Complex, from v2f64: legal type is v2i32, 1 narrowing => ~2.
+      {ISD::FP_TO_SINT, MVT::v2i32, MVT::v2f64, 2},
+      {ISD::FP_TO_SINT, MVT::v2i16, MVT::v2f64, 2},
----------------
sdesmalen wrote:
> Please avoid changing the formatting of the table, because that makes this diff unnecessarily big.
I believe this was a clang-format change.


================
Comment at: llvm/test/CodeGen/AArch64/fptrunc-cost.ll:1
+; RUN: opt -debug-only=loop-vectorize -loop-vectorize -mtriple aarch64-linux-gnu -mattr=+sve -S -o - < %s  2>&1| FileCheck %s
+
----------------
sdesmalen wrote:
> The test is in the wrong directory. SVE Cost-model tests should be added in `llvm/test/Analysis/CostModel/AArch64`
Ah my mistake, will fix!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98934