[llvm-tapi] Don't try to override SequenceTraits for std::string
authorSam Clegg <sbc@chromium.org>
Fri, 7 Dec 2018 19:29:00 +0000 (19:29 +0000)
committerSam Clegg <sbc@chromium.org>
Fri, 7 Dec 2018 19:29:00 +0000 (19:29 +0000)
commit992fc88111578d04b21ed1f9c5ade82a502423d5
treecd3dd018cbb53c186dd65f6d122e6d68d75fdba6
parent48c7b4f0b6ca5dcaaf388e1c7aa90ec72ba4c6b0
[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

llvm-svn: 348630
llvm/lib/TextAPI/ELF/TBEHandler.cpp
llvm/unittests/TextAPI/ELFYAMLTest.cpp