[all-commits] [llvm/llvm-project] b7a112: [LoopVectorize] Fix scalarisation crash in widenPH...
david-arm via All-commits
all-commits at lists.llvm.org
Wed May 12 03:02:36 PDT 2021
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: b7a11274f90f07537e2151fa4424db257ff9a950
https://github.com/llvm/llvm-project/commit/b7a11274f90f07537e2151fa4424db257ff9a950
Author: David Sherwood <david.sherwood at arm.com>
Date: 2021-05-12 (Wed, 12 May 2021)
Changed paths:
M llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
M llvm/test/Transforms/LoopVectorize/AArch64/sve-widen-phi.ll
Log Message:
-----------
[LoopVectorize] Fix scalarisation crash in widenPHIInstruction for scalable vectors
In InnerLoopVectorizer::widenPHIInstruction there are cases where we have
to scalarise a pointer induction variable after vectorisation. For scalable
vectors we already deal with the case where the pointer induction variable
is uniform, but we currently crash if not uniform. For fixed width vectors
we calculate every lane of the scalarised pointer induction variable for a
given VF, however this cannot work for scalable vectors. In this case I
have added support for caching the whole vector value for each unrolled
part so that we can always extract an arbitrary element. Additionally, we
still continue to cache the known minimum number of lanes too in order
to improve code quality by avoiding an extractelement operation.
I have adapted an existing test `pointer_iv_mixed` from the file:
Transforms/LoopVectorize/consecutive-ptr-uniforms.ll
and added it here for scalable vectors instead:
Transforms/LoopVectorize/AArch64/sve-widen-phi.ll
Differential Revision: https://reviews.llvm.org/D101294
More information about the All-commits
mailing list