[PATCH] D113973: [LoopVectorize][CostModel] Choose smaller VFs for in-loop reductions with no loads/stores

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.

LGTM, thanks!



================
Comment at: llvm/include/llvm/Analysis/IVDescriptors.h:257
+  /// Returns the minimum width used by the recurrence.
+  unsigned getMinWidthCastToRecurrenceType() const {
+    return MinWidthCastToRecurrenceType;
----------------
it would be good to add `InBits` or something to make clear the value is in bits.


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

https://reviews.llvm.org/D113973