[PATCH] D73940: GlobalISel: Reimplement moreElementsVectorDst

Petar Avramovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 6 07:58:03 PST 2020


Petar.Avramovic added a comment.

This looks like a good candidate for new `LegalizeAction` ( MoreElementsLCM or something more appropriate ) that could be preferable more elements option for targets that support `G_CONCAT_VECTORS`  and `G_UNMERGE_VALUES` for large vector sizes (large enough to hold `LCM(WideTy, OrigTy)` ).
Current algorithm seems to be designed for targets that prefer not to deal with vector types larger then WideTy. It would be difficult for them to deal with `G_CONCAT_VECTORS`  and `G_UNMERGE_VALUES` that are not legal and likely hard to combine since they have different small LLTs.


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

https://reviews.llvm.org/D73940





More information about the llvm-commits mailing list