diff options
author | Hans Wennborg <hans@hanshq.net> | 2019-06-27 13:55:02 +0000 |
---|---|---|
committer | Hans Wennborg <hans@hanshq.net> | 2019-06-27 13:55:02 +0000 |
commit | 408fc0849ea1f630baa85d5bf78ee359c52585e1 (patch) | |
tree | dbfa29e95e14e44ac8b3760553a47285cf73d5d5 /llvm/lib/Bitcode/Writer/BitcodeWriter.cpp | |
parent | a0d45058ebb39dc18cbcdcba01d11d36eca001be (diff) | |
download | llvm-408fc0849ea1f630baa85d5bf78ee359c52585e1.zip llvm-408fc0849ea1f630baa85d5bf78ee359c52585e1.tar.gz llvm-408fc0849ea1f630baa85d5bf78ee359c52585e1.tar.bz2 |
Revert r363658 "[SVE][IR] Scalable Vector IR Type with pr42210 fix"
We saw a 70% ThinLTO link time increase in Chromium for Android, see
crbug.com/978817. Sounds like more of PR42210.
> Recommit of D32530 with a few small changes:
> - Stopped recursively walking through aggregates in
> the verifier, so that we don't impose too much
> overhead on large modules under LTO (see PR42210).
> - Changed tests to match; the errors are slightly
> different since they only report the array or
> struct that actually contains a scalable vector,
> rather than all aggregates which contain one in
> a nested member.
> - Corrected an older comment
>
> Reviewers: thakis, rengolin, sdesmalen
>
> Reviewed By: sdesmalen
>
> Differential Revision: https://reviews.llvm.org/D63321
llvm-svn: 364543
Diffstat (limited to 'llvm/lib/Bitcode/Writer/BitcodeWriter.cpp')
-rw-r--r-- | llvm/lib/Bitcode/Writer/BitcodeWriter.cpp | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp b/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp index 1cf55ba..1f39fa3 100644 --- a/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp +++ b/llvm/lib/Bitcode/Writer/BitcodeWriter.cpp @@ -938,13 +938,10 @@ void ModuleBitcodeWriter::writeTypeTable() { } case Type::VectorTyID: { VectorType *VT = cast<VectorType>(T); - // VECTOR [numelts, eltty] or - // [numelts, eltty, scalable] + // VECTOR [numelts, eltty] Code = bitc::TYPE_CODE_VECTOR; TypeVals.push_back(VT->getNumElements()); TypeVals.push_back(VE.getTypeID(VT->getElementType())); - if (VT->isScalable()) - TypeVals.push_back(VT->isScalable()); break; } } |