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

[Rosie Sumpter via Phabricator via llvm-commits]

RosieSumpter marked an inline comment as done.
RosieSumpter added inline comments.


================
Comment at: llvm/include/llvm/Analysis/IVDescriptors.h:257
+  /// Returns the minimum width used by the recurrence.
+  unsigned getMinWidthCastToRecurrenceType() const {
+    return MinWidthCastToRecurrenceType;
----------------
fhahn wrote:
> it would be good to add `InBits` or something to make clear the value is in bits.
I made this change before committing. Thanks for your help with this patch :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113973