[PATCH] D80640: [CodeGen][SVE] Calculate correct type legalization for scalable vectors.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 28 13:45:45 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetLoweringBase.cpp:1358
+            // that don't support scalable vectors can't lower them at all and
+            // end up here.
       } else {
----------------
sdesmalen wrote:
> efriedma wrote:
> > We probably want a new LegalizeTypeAction kind to represent this.  I guess just leaving it TypeLegal isn't too terrible for now, though.
> Something like `TypeCannotBeLegalized`?
I was thinking something more like "TypeScalarizeVScaleVector".  It's theoretically possible to legalize... although it's complicated by the fact that we can't easily emit loops in SelectionDAG.


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

https://reviews.llvm.org/D80640





More information about the llvm-commits mailing list