[PATCH] D85794: [WIP][llvm][LV] Replace `unsigned VF` with `ElementCount VF` [NFCI]

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli added a comment.

In D85794#2213912 <https://reviews.llvm.org/D85794#2213912>, @rogfer01 wrote:

>> Out of curiosity, what is your approach? Carrying around a `bool Scalable` flag?
>
> We keep some state around that tells us if we're in a scalable mode. But this doesn't work if we want to be able to vectorize both for fixed size or scalable at the same time for the same target. Not sure if you're considering this scenario (e.g. Advanced SIMD + SVE-256)

I don't see why we should exclude this scenario. Better to keep the possibilities open for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85794