[PATCH] D94913: [SLP] match maxnum/minnum intrinsics as FP reduction ops

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 18 08:51:47 PST 2021


ABataev added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7106-7108
+          TTI->getIntrinsicInstrCost(VecAttr, TTI::TCK_RecipThroughput);
+      ScalarCost =
+          TTI->getIntrinsicInstrCost(ScaAttr, TTI::TCK_RecipThroughput);
----------------
Probably better to use getMinMaxReductionCost rather than the cost of the intrinsic call, it is more precise.


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

https://reviews.llvm.org/D94913



More information about the llvm-commits mailing list