<div dir="ltr">If it's not correct in the object file I think the best fix is to make it correct in the object file.  I actually had to fix a few things like that in the past.  Either way, don't worry about it.  I will get around to it eventually :)  Maybe a comment saying // TODO: The arch in the object file isn't correct for MSVC binaries on windows, so we should find a way to make it correct.<br></div><br><div class="gmail_quote">On Thu, May 14, 2015 at 2:05 PM Robert Flack <<a href="mailto:flackr@gmail.com">flackr@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">REPOSITORY<br>
  rL LLVM<br>
<br>
================<br>
Comment at: lldb/trunk/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp:4054-4055<br>
@@ -4054,4 +4053,4 @@<br>
                             ConstString demangled_name = sc.GetFunctionName(Mangled::ePreferDemangled);<br>
                             if (strncmp(mangled_name.GetCString(), "_ZN", 3) ||<br>
                                 !strncmp(demangled_name.GetCString(), "(anonymous namespace)", 21))<br>
                             {<br>
----------------<br>
zturner wrote:<br>
> This check is only meaningful under a particular ABI.  Could you check the triple or ArchSpec of the target you're looking at first, and either not do this if the triple contains MSVC (or better yet, add a correct check)?<br>
Hey, I added a check that the object file is linux or freebsd: <a href="http://reviews.llvm.org/D9782" target="_blank">http://reviews.llvm.org/D9782</a>. I tried to add a check for windows arch as well, but the arch was not correctly recognized in my tests, I was seeing i386-unknown-unknown. Let me know if you know where I might be able to find the target arch instead if not through the object file and I'm happy to add a correct Windows check as well. Thanks.<br>
<br>
<a href="http://reviews.llvm.org/D9754" target="_blank">http://reviews.llvm.org/D9754</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</blockquote></div>