[PATCH] D13998: Move parts of llvm-symbolizer tool into LLVMSymbolize library.

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 19:16:08 PDT 2015


On Fri, Oct 23, 2015 at 7:11 PM Rafael EspĂ­ndola <
llvm-commits at lists.llvm.org> wrote:

>
> On Oct 23, 2015 8:40 PM, "Eric Christopher" <echristo at gmail.com> wrote:
> >
> > echristo added a reviewer: rafael.
> > echristo added a comment.
> >
> > Couple of inline comments otherwise it looks fine to me.
> >
> > Rafael: You had objected originally about this, do you still mind?
>
> I don't think we should move code just to use gtest. If you have a use for
> the library feel free to move the code.
>
>
Definitely have a use for the library in other applications. Mike's work
will be good that way.

Thanks!

-eric


> Cheers, Rafael
> _______________________________________________
> 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/20151024/5c5780e6/attachment.html>


More information about the llvm-commits mailing list