<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 27, 2017 at 4:54 PM, Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Moving the itanium demangler to Common makes sense since it is used by<br>
various formats and can be built everywhere.<br>
<br>
I am not sure of the value of moving the msvc demangler. It is only used<br>
by COFF and can only be built on windows.<br></blockquote><div><br></div><div>Moving the MSVC demangler wrapper is for symmetry. Before this patch, we have `lld::coff::demangle` and `lld::elf::demangle`. Just renaming lld::elf::demangle lld::demangle is ambiguous, so I had to rename it lld::demangleItanium. Now we have several choices on what to do to the MSVC demangler.</div><div><br></div><div>  1. lld::demangleItanium and lld::coff::demangle</div><div>  2. lld::demangleItanium and lld::coff::demangleMSVC</div><div>  3. lld::demangleItanium and lld::demangleMSVC</div><div><br></div><div>The first two choices feel a bit inconsistent, so I chose the last one.</div></div></div></div>