[PATCH] D144474: [LV][NFC] Use ElementCount for getMaxInterleaveFactor

[Luke Lau via Phabricator via llvm-commits]

luke added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h:276
     // Let regular unroll to unroll the loop.
-    return VF == 1 ? 1 : ST->getMaxInterleaveFactor();
+    return VF.getKnownMinValue() == 1 ? 1 : ST->getMaxInterleaveFactor();
   }
----------------
fhahn wrote:
> Shouldn't that be `VF.isScalar()`?
I plan on doing this in a follow up patch to keep this one NFC.
Disabling interleaving for scalable vectors changes a good few RISC-V test cases


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144474