[PATCH] D48175: [BinaryFormat] Add MsgPackTypes

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 28 14:54:16 PDT 2018


dblaikie added a comment.

In https://reviews.llvm.org/D48175#1216609, @scott.linder wrote:

> I've updated the patch to just support shared_ptr; I think the more generic option is right for now.


I'd still be curious to understand the usage you're picturing that creates the requirement for shared ownership, if you could paint a bit of a picture around that, that'd be super handy!


https://reviews.llvm.org/D48175





More information about the llvm-commits mailing list