[PATCH] D23213: Allow SmallVector to be used as a yaml sequence

[Daniel Berlin via llvm-commits]

Yeah.
I think you should give up on the friends and make it public or something.

I could not find a set of friend definitions that make it work.



On Fri, Aug 5, 2016 at 4:01 PM, Zachary Turner <zturner at google.com> wrote:

> I'm not sure why this doesn't break with cl or clang-cl.  In any case,
> those friend definitions are a pretty ugly hack :-/
>
> On Fri, Aug 5, 2016 at 3:28 PM Daniel Berlin <dberlin at dberlin.org> wrote:
>
>> The friend definitions in DebugMap.h need to be updated now that it's
>> SmallVector and not std::vector
>>
>> On Fri, Aug 5, 2016 at 3:22 PM, Oleksiy Vyalov via llvm-commits <
>> llvm-commits at lists.llvm.org> wrote:
>>
>>> ovyalov added a subscriber: ovyalov.
>>> ovyalov added a comment.
>>>
>>> Looks like this CL broke LLDB cmake buildbot - http://lab.llvm.org:8011/
>>> builders/lldb-x86_64-ubuntu-14.04-cmake/builds/17873/
>>> steps/ninja%20build%20local/logs/stdio, could you take a look?
>>>
>>>
>>> Repository:
>>>   rL LLVM
>>>
>>> https://reviews.llvm.org/D23213
>>>
>>>
>>>
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>>
>>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160805/b732fc7c/attachment.html>