[PATCH] D51174: Move Microsoft Demangler to Support

[Nico Weber via Phabricator via llvm-commits]

thakis added a comment.

In https://reviews.llvm.org/D51174#1211835, @zturner wrote:

> 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?


I copy over the source files I need, so .lib files wouldn't help me. Right now, it's "copy 1 file" and it's fairly easy to rev. It'd become "figure out subset of Support currently needed", which will likely be different each time. Keeping things in their own project does keep this more manageable (which is why this how it's done for the itanium mangler).


https://reviews.llvm.org/D51174