[PATCH] Add std::string to YAML I/O built-in types

Sean Silva silvas at purdue.edu
Thu Oct 24 17:03:25 PDT 2013


On Thu, Oct 24, 2013 at 11:35 AM, John Thompson <
john.thompson.jtsoftware at gmail.com> wrote:

>
>   It broke a number of builds, so I'm taking a more local approach with a
> local mapping.
>

What exactly was the issue with introducing this functionality? I can't
imagine that there would be a fundamental issue with doing this (e.g.,
there might be some issues where the client code needs to be tightened up,
but they can just be fixed up at the usage sites).

I really don't think a local mapping is the right approach (unless there's
some truly fundamental issue with having this specialization).

-- Sean Silva


>
> http://llvm-reviews.chandlerc.com/D2006
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131024/4d8c1225/attachment.html>


More information about the llvm-commits mailing list