
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Sep 4, 2017, at 12:10 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><br class=""><br class=""><div class="gmail_quote"><div dir="ltr" class="">On Thu, Aug 31, 2017 at 2:18 PM Reid Kleckner via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" class="">llvm-commits@lists.llvm.org</a>> wrote:<br class=""></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk added a comment.<br class="">

<br class="">

Running debuginfo-tests was enlightening. My understanding of these opcodes is a lot worse than I thought it was. :( They are really impossible to understand without staring at the LLDB or GDB expression evaluation code. I'll have to think about this some more.<br class=""></blockquote><div class=""><br class="">Happy to chat about them in person if it helps. There was also some historic misunderstanding of some of the opcodes that ended up baked into LLDB and LLVM (they both agreed on something but it wasn't the thing the DWARF Spec said, then we changed it somehow (this was related to DW_OP_bit_piece))<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>Yes, I fixed this in both LLVM and LLDB. The debuginfo-tests predate DW_OP_bit_piece support in LLVM significantly, so the locations in there are a lot simpler. Since the tests are run with both GDB and LLDB, they should be a little better guarded against symmetric errors like that, although I am not sure whether there is a current bot running debuginfo-tests with GDB at the moment.</div><div><br class=""></div><div>-- adrian</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_quote"><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br class="">

<br class="">

<a href="https://reviews.llvm.org/D37311" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/D37311</a><br class="">

<br class="">

<br class="">

<br class="">

_______________________________________________<br class="">

llvm-commits mailing list<br class="">

<a href="mailto:llvm-commits@lists.llvm.org" target="_blank" class="">llvm-commits@lists.llvm.org</a><br class="">

<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank" class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br class="">

</blockquote></div></div>

</div></blockquote></div><br class=""></body></html>