[llvm-branch-commits] [llvm] [LV][REVEC][AArch64] Proof of concept for re-vectorisation (PR #208213)

[Gaëtan Bossu via llvm-branch-commits]

================
@@ -279,7 +282,9 @@ ElementCount VFSelectionContext::getMaximizedVFForTarget(
   else
     MaxPermissibleVFWithoutMaxBW.FixedVF = MaxVF;
 
-  if (useMaxBandwidth(ComputeScalableMaxVF)) {
+  // REVEC: Avoid creating wider than expected scalable types by choosing
+  // VF > vscale x 1.
+  if (useMaxBandwidth(ComputeScalableMaxVF) && !Legal->LoopContainsVectors) {
----------------
gbossu wrote:

It shouldn't be. It should be up to the cost model to decide what the best VF is. I'll get rid of this.

https://github.com/llvm/llvm-project/pull/208213