[PATCH] D86995: [VectorCombine][SVE] Do not fold bitcast shuffle for scalable type.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 2 06:00:00 PDT 2020


spatel added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VectorCombine.cpp:446-447
 
   // FIXME: it should be possible to implement the computation of the widened
   // shuffle mask in terms of ElementCount to work with scalable shuffles.
   unsigned DestNumElts = cast<FixedVectorType>(DestTy)->getNumElements();
----------------
spatel wrote:
> Move this comment to up to the point that we are excluding ScalableVectorType.
(or remove if it is not possible)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86995



More information about the llvm-commits mailing list