[Lldb-commits] [PATCH] Change lldb_assert() to use llvm::sys::PrintBacktrace

Ed Maste emaste at freebsd.org
Wed Mar 4 17:53:13 PST 2015


On 4 March 2015 at 20:36, Zachary Turner <zturner at google.com> wrote:
>
> This seems like kind of a straightforward case of code reuse to
> me, and a clear win, so I'm not sure why it's that contentious.
>
> If we need to interoperate with StreamString, then we should implement an
> overload **in LLVM** that returns an std::string.

I agree completely. For OS X, having our own LLDB-specific
implementation may not make too much difference. But the downsides on
other platforms outweigh whatever benefit we might get from having it.



More information about the lldb-commits mailing list