[llvm-tapi] Don't try to override SequenceTraits for std::string
commita97d9f8bb5bc56ad224161d90411146cef5a3d13
authorSam Clegg <sbc@chromium.org>
Fri, 7 Dec 2018 19:29:00 +0000 (7 19:29 +0000)
committerSam Clegg <sbc@chromium.org>
Fri, 7 Dec 2018 19:29:00 +0000 (7 19:29 +0000)
tree4055a53e8ddedf085c285b9c3bb0b0b2e2793d7f
parentd19f12b58f11ce8a668ba7ffa43b5c3d883e3588
[llvm-tapi] Don't try to override SequenceTraits for std::string

For some reason this doesn't seem to work with LLVM_LINK_LLVM_DYLIB
build.

See https://logs.chromium.org/logs/chromium/bb/client.wasm.llvm/linux/37764/+/recipes/steps/LLVM_regression_tests/0/stdout

What is more it seems that overriding these traits for core types
(including std::string) is not supported/recommend by YAMLTraits.h.
See line 1918 which has the assertion:
 "only use LLVM_YAML_IS_SEQUENCE_VECTOR for types you control"

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@348630 91177308-0d34-0410-b5e6-96231b3b80d8
lib/TextAPI/ELF/TBEHandler.cpp
unittests/TextAPI/ELFYAMLTest.cpp