<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 21, 2017 at 4:05 PM Greg Clayton 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">clayborg accepted this revision.<br>
clayborg added a comment.<br>
<br>
Looks like there already is a test case that was failing as Jim mentioned. Accepting based on that.<br></blockquote><div><br>Ah, I was thinking more a test that would've failed when LLDB regressed (regardless of whether Clang was still producing this DWARF or not) - does LLDB have tests like that? (either binary, asm, or some other terse way of writing DWARF directly to test "does LLDB do the right thing with this DWARF" sort of tests?)<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="https://reviews.llvm.org/D35740" rel="noreferrer" target="_blank">https://reviews.llvm.org/D35740</a><br>
<br>
<br>
<br>
</blockquote></div></div>