[PATCH] D44735: [LV] Add TTI::shouldMaximizeVectorBandwidth to allow enabling it per target

[Krzysztof Parzyszek via Phabricator via llvm-commits]

kparzysz added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:1344
   }
+  bool shouldMaximizeVectorBandwidth(bool OptSize) const override {
+    return Impl.shouldMaximizeVectorBandwidth(OptSize);
----------------
craig.topper wrote:
> Is const needed here? Looking at the nearby methods, it seems only getRegisterBitWidth has it.
No, it's not needed.  Should I remove it?  What is the best practice here?


Repository:
  rL LLVM

https://reviews.llvm.org/D44735