[PATCH] D146033: [AArch64][TTI] Cost model FADD/FSUB/FNEG

[Sjoerd Meijer via Phabricator via llvm-commits]

SjoerdMeijer added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:2420
+    if (!Ty->getScalarType()->isFP128Ty())
+      return LT.first;
   case ISD::FMUL:
----------------
MaskRay wrote:
> Also an `error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]` to address before you reland the patch :)
Included/fixed in the recommit:
https://reviews.llvm.org/rGd827865e9f77


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146033