[PATCH] D55381: [llvm-tapi] Don't try to override SequenceTraits for std::string

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 6 12:08:19 PST 2018


sbc100 added a comment.

In D55381#1321857 <https://reviews.llvm.org/D55381#1321857>, @amontanez wrote:

> Apologies, I'll keep a closer eye on that in the future. This can be resolve by doing what ELFArch has done. This requires adding a typedef to TBEHandler.h, changing the declaration of NeededLibs to use the new typedef, and then modifying the ELFYAMLTest.cpp to use something like
>
>   LLVM_YAML_STRONG_TYPEDEF(ELFNeededList, ELFNeededMapper)
>
>
> That should allow us to keep the flow mapping.


Nice!  Do you want to do that or should I?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55381/new/

https://reviews.llvm.org/D55381





More information about the llvm-commits mailing list