Revert r363658 "[SVE][IR] Scalable Vector IR Type with pr42210 fix"
authorHans Wennborg <hans@hanshq.net>
Thu, 27 Jun 2019 13:55:02 +0000 (13:55 +0000)
committerHans Wennborg <hans@hanshq.net>
Thu, 27 Jun 2019 13:55:02 +0000 (13:55 +0000)
commit408fc0849ea1f630baa85d5bf78ee359c52585e1
treedbfa29e95e14e44ac8b3760553a47285cf73d5d5
parenta0d45058ebb39dc18cbcdcba01d11d36eca001be
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
19 files changed:
llvm/docs/LangRef.rst
llvm/include/llvm/ADT/DenseMapInfo.h
llvm/include/llvm/IR/DerivedTypes.h
llvm/include/llvm/IR/Type.h
llvm/include/llvm/Support/ScalableSize.h [deleted file]
llvm/lib/AsmParser/LLLexer.cpp
llvm/lib/AsmParser/LLParser.cpp
llvm/lib/AsmParser/LLToken.h
llvm/lib/Bitcode/Reader/BitcodeReader.cpp
llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
llvm/lib/IR/AsmWriter.cpp
llvm/lib/IR/LLVMContextImpl.h
llvm/lib/IR/Type.cpp
llvm/lib/IR/Verifier.cpp
llvm/test/Bitcode/compatibility.ll
llvm/test/Verifier/scalable-aggregates.ll [deleted file]
llvm/test/Verifier/scalable-global-vars.ll [deleted file]
llvm/unittests/IR/CMakeLists.txt
llvm/unittests/IR/VectorTypesTest.cpp [deleted file]