I think just remove the preprocessor block.  Imagine debugging a Windows target from a linux host.  This function would return false as written because the host wasn’t Windows <br><div class="gmail_quote"><div dir="ltr">On Thu, Feb 8, 2018 at 12:18 PM Aaron Smith via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">asmith added a comment.<br>
<br>
IsCPPMangledName() is a public method to determine if a symbol has a mangled name.<br>
<br>
This change is needed so that lldb can find symbols with MSVC style mangled names.<br>
<br>
Open to suggestions on a better way to deal with this...<br>
<br>
Here are all the places it is used:<br>
<br>
source/Core/Module.cpp:    if (CPlusPlusLanguage::IsCPPMangledName(name_cstr))<br>
source/Core/Module.cpp:          !CPlusPlusLanguage::IsCPPMangledName(name_cstr)) {<br>
source/Core/Mangled.cpp:      if (CPlusPlusLanguage::IsCPPMangledName(mangled_name))<br>
source/Plugins/ExpressionParser/Clang/ClangExpressionDeclMap.cpp:                           !CPlusPlusLanguage::IsCPPMangledName(name)) ||<br>
source/Expression/IRExecutionUnit.cpp:    if (CPlusPlusLanguage::IsCPPMangledName(name.GetCString())) {<br>
source/Expression/IRExecutionUnit.cpp:    if (CPlusPlusLanguage::IsCPPMangledName(name.GetCString())) {<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D43059" rel="noreferrer" target="_blank">https://reviews.llvm.org/D43059</a><br>
<br>
<br>
<br>
</blockquote></div>