[PATCH] D105432: [Analysis] Add simple cost model for strict (in-order) reductions

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 9 08:26:49 PDT 2021


RKSimon added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:1181
+  ///   operations in lane order, starting with an initial scalar value, i.e.
+  ///     result = InitVal + v0 + v1 + v2 + v3
+  enum class ReductionType : uint8_t { TreeWise, Ordered };
----------------
add explicit parentheses to make this even more obvious?

result = (((InitVal + v0) + v1) + v2) + v3


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

https://reviews.llvm.org/D105432



More information about the llvm-commits mailing list