<div dir="ltr">Thanks, created here: <a href="https://reviews.llvm.org/D40283">https://reviews.llvm.org/D40283</a><div><br></div><div>I got here via <a href="http://llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch">http://llvm.org/docs/DeveloperPolicy.html#making-and-submitting-a-patch</a> ; Should that documentation be updated? Would a patch be appropriate?</div><div><br></div><div>- Nelson</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 20, 2017 at 7:57 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Please submit a review of this patch through <a href="http://reviews.llvm.org" target="_blank">reviews.llvm.org</a>. <div><br></div><div>- svn diff -x -U99999 > /tmp/a.patch</div><div>- login or create a login if you don't already have one</div><div>- Select Differential from the top left corner</div><div>- Click "+ Create Diff" in upper right</div><div>- Click "Choose File" and select /tmp/a.patch, no need to fill anything in the Raw Diff field or Repository then click the "Create Diff" button</div><div>- Review your code and click the button to continue</div><div>- Fill in title, description and add the names of reviewers and add lldb-commits to the reviewers</div><div><br></div><div><div><blockquote type="cite"></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div>On Nov 19, 2017, at 10:33 PM, Nelson Elhage via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a>> wrote:</div><br class="m_-3031806498819144770Apple-interchange-newline"></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div><div dir="ltr">When importing C++ methods into clang AST nodes from the DWARF symbol table, preserve the DW_AT_linkage_name and use it as the linker ("asm") name for the symbol.<div><br></div><div>Concretely, this enables `expression` to call into names that use the GNU `abi_tag` extension, and enables lldb to call into code using std::string or std::list from recent versions of libstdc++. See <a href="https://bugs.llvm.org/show_bug.cgi?id=35310" target="_blank">https://bugs.llvm.org/show_bug.cgi?id=35310</a> . It also seems broadly more robust than relying on the DWARF->clang->codegen pipeline to roundtrip properly, but I'm not immediately aware of any other cases in which it makes a difference.</div><div><br></div><div>- Nelson</div></div>
</div></blockquote></div></div></div><div style="word-wrap:break-word"><div><div><blockquote type="cite"><div><span id="m_-3031806498819144770cid:15fd81fd1873db6acb71"><bug35310.diff></span>_______________________________________________<br>lldb-commits mailing list<br><a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br></div></blockquote></div><br></div></div></blockquote></div>