[PATCH] D44490: [AArch64] Implement getArithmeticReductionCost

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 15 02:23:45 PDT 2018


samparker accepted this revision.
samparker added a comment.
This revision is now accepted and ready to land.

I'm happy to see this! LGTM with just one small comment, no need to re-review. Thanks!



================
Comment at: lib/Target/AArch64/AArch64TargetTransformInfo.cpp:911
+
+  return BaseT::getArithmeticReductionCost(Opcode, ValTy, IsPairwise);
+}
----------------
You could perform the IsPairwise check at the beginning of the function to exit early.


Repository:
  rL LLVM

https://reviews.llvm.org/D44490





More information about the llvm-commits mailing list