HomePhabricator

[SLPVectorizer][SVE] Bail out early for scalable vector.

Authored by huihuiz on Mar 13 2020, 11:23 AM.

Description

[SLPVectorizer][SVE] Bail out early for scalable vector.

Summary:
SLPVectorizer try to vectorize list of scalar instructions of the same type,
instructions already vectorized are rejected through isValidElementType().

Without this patch, tryToVectorizeList() will first try to determine vectorization
factor of a list of Instructions before checking whether each instruction has unsupported
type or not. For instructions already vectorized for SVE, it will crash at getVectorElementSize(),
where it try to return a fixed size.

This patch make sure invalid element types are rejected before trying to get vectorization
factor. This make sure we are not trying to vectorize instructions already vectorized.

Reviewers: sdesmalen, efriedma, spatel, RKSimon, ABataev, apazos, rengolin

Reviewed By: efriedma

Subscribers: tschuett, hiraditya, rkruppe, psnobl, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D76017

Details

Committed
huihuizMar 13 2020, 11:23 AM
Reviewer
efriedma
Differential Revision
D76017: [SLPVectorizer][SVE] Bail out early for scalable vector.
Parents
rG842ea709e4ed: Debug Info: Store the SDK in the DICompileUnit.
Branches
Unknown
Tags
Unknown