[PATCH] D51174: Move Microsoft Demangler to Support

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 23 17:37:11 PDT 2018


What if we produced a .lib file that had only the minimal set of object
files needed to link this, and make MicrosoftDemangle.h not include other
llvm headers, then you could just link against that object file and it
should be effectively independent.

Would that work?
On Thu, Aug 23, 2018 at 5:32 PM Nico Weber via Phabricator <
reviews at reviews.llvm.org> wrote:

> thakis added a comment.
>
> FWIW I'd like to use the new demangler in demumble [1], but if I need to
> pull in most of Support of it then I probably won't do that. I agree this
> shouldn't impact LLVM much (or at all), but having stand-alone-ish
> demangler code is probably nice for several other projects too. So since
> the itanium demangler is still in the other place and likely will stay
> there, maybe it's nice to keep the ms demangler next to it. (But as I said,
> don't weigh this feedback heavily.)
>
> 1: https://github.com/nico/demumble
>
>
> https://reviews.llvm.org/D51174
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180823/1f94865c/attachment.html>


More information about the llvm-commits mailing list