diff options
author | Cullen Rhodes <cullen.rhodes@arm.com> | 2020-03-18 16:51:45 +0000 |
---|---|---|
committer | Cullen Rhodes <cullen.rhodes@arm.com> | 2020-03-19 10:08:17 +0000 |
commit | 5c296df0c0a4dea364c17bb1d7bc5360c3ce36fd (patch) | |
tree | ce5a49eff60669319eeb8198bcc78595a020af26 /clang/lib/Frontend/CompilerInvocation.cpp | |
parent | 6e0aaafbc782bdd2852a48c87a3f6cc8fa667bee (diff) | |
download | llvm-5c296df0c0a4dea364c17bb1d7bc5360c3ce36fd.zip llvm-5c296df0c0a4dea364c17bb1d7bc5360c3ce36fd.tar.gz llvm-5c296df0c0a4dea364c17bb1d7bc5360c3ce36fd.tar.bz2 |
[ValueTypes] Add EVT::isFixedLengthVector
Summary:
Related to D75672, this patch adds EVT::isFixedLengthVector to determine
if the underlying vector type is of fixed length.
An assert is introduced in EVT::getVectorNumElements that triggers for
types that aren't fixed length. This is currently guarded by a flag
added D75297 that is off by default and has been renamed to the more
generic ENABLE_STRICT_FIXED_SIZE_VECTORS.
Ideally we want to get rid of getVectorNumElements but a quick grep
shows there are >350 uses in lib/CodeGen and 75 in lib/Target/AArch64
alone. All of these probably aren't EVT::getVectorNumElements (some may
be the MVT equivalent), but there are many places to fixup and having
the assert on by default would make the SVE upstreaming effort
difficult.
Reviewers: sdesmalen, efriedma, ctetreau, huntergr, rengolin
Reviewed By: efriedma
Subscribers: mgorny, kristof.beyls, hiraditya, danielkiss, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D76376
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions