[PATCH] D34907: fix ODR violations due to abuse of LLVM_YAML_IS_(FLOW_)?SEQUENCE_VECTOR

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 30 13:51:34 PDT 2017


zturner accepted this revision.
zturner added a comment.
This revision is now accepted and ready to land.

I don't have any comments, this seems fine to me.  Long term I think the best solution is to get rid of these global specializations entirely, and instead provide adapters called `flow` and `block`, and instead of just saying `IO.mapOptional("Foo", Sequence);` and having it be picked up by a specialization, you would say `IO.mapOptional("Foo", flow(Sequence));`


Repository:
  rL LLVM

https://reviews.llvm.org/D34907





More information about the llvm-commits mailing list