<div dir="ltr"><div>These two changes (<a href="https://github.com/vadimcn/llvm/commit/a8e52b43f4ca43594987ad900277bed7c005461b" target="_blank">[1]</a> <a href="https://github.com/vadimcn/lldb/commit/77d54fcbcb82b0b94bcfb238387b055489f3632e" target="_blank">[2]</a>) appear to be sufficient to enable debugging of Rust binaries with pdb debug info (make sure that msdia140.dll is somewhere lldb can find it).</div><div></div><div></div><div></div><div>At a cursory glance, I did not observe any ill effects of disabling that language check, so I'm still not sure whether it was truly necessary, or more of "Let's disable code paths we did not test".<br></div><div><br></div><div>As for allocating a language code for Rust, does Microsoft even have a procedure for this?   In any case, looks like D folks had already <a href="https://github.com/llvm-mirror/llvm/blob/d7e8f44ac5abf9599b718cf1438ffbe8adee5db9/include/llvm/DebugInfo/CodeView/CodeView.h#L165" target="_blank">staked out</a> 0x44 for themselves.   I suggest we claim 0x27  :-)<br></div><div></div><div><div><br></div><div class="gmail_quote"><div dir="ltr">On Mon, Aug 27, 2018 at 6:32 AM Tom Tromey <<a href="mailto:ttromey@mozilla.com" target="_blank">ttromey@mozilla.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">>>>>> "Vadim" == Vadim Chugunov via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a>> writes:<br>
<br>
Vadim> Would you mind going into a bit more detail on what sort of<br>
Vadim> problems an unknown language could cause?  I'd like to understand<br>
Vadim> the issue before jumping in to fix anything.  AFAIK, in the case<br>
Vadim> of DWARF symbols, debug info for unknown languages is still used,<br>
Vadim> so it wouldn't be the first for LLDB...<br>
<br>
LLDB checks for DW_LANG_Rust and uses the C++ plugin in this case.<br>
<br>
The Rust lldb (<a href="https://github.com/rust-lang-nursery/lldb/commits/rust-release-70" rel="noreferrer" target="_blank">https://github.com/rust-lang-nursery/lldb/commits/rust-release-70</a>)<br>
removes this.<br>
<br>
If you get something working, let me know.  I'd like to incorporate it<br>
into the Rust lldb.<br>
<br>
Tom<br>
</blockquote></div></div></div>