<div dir="ltr">Ping</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 28, 2017 at 8:20 PM David Blaikie via Phabricator via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">dblaikie added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/trunk/test/tools/llvm-dwarfdump/X86/brief.s:7-10<br>
 # CHECK: DW_AT<br>
+# CHECK-NOT: debug_str<br>
+# CHECK-NOT: DW_AT_type {{.*}} =><br>
<br>
----------------<br>
It's probably worth having a positive test for what the string printing looks like in brief mode? I know it was mentioned in the patch description, but it was revised before committing & I was curious to see how it actually prints out now.<br>
<br>
FWIW, I'd think we might want to drive towards printing something as simple as:<br>
<br>
  compile_unit<br>
    name: "foo"<br>
<br>
(ie: dropping as much of the ()[]{} sort of stuff as possible, and dropping the DW_*_ prefixes too - since they're unambiguous as to what's a DW_TAG, DW_AT, DW_OP, etc, based on context)<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D36835" rel="noreferrer" target="_blank">https://reviews.llvm.org/D36835</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>