<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Feb 1, 2018 at 3:37 PM Paul Robinson 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">probinson added a comment.<br>
<br>
@JDevlieghere are the quoted file/directory names compatible (enough) with Darwin's dwarfdump?  I know you were working to make them compatible (although maybe it was mostly command-line compatible).<br>
<br>
<br>
<br>
================<br>
Comment at: llvm/test/DebugInfo/X86/dwarfdump-header.s:327-328<br>
 # CHECK: max_ops_per_inst: 1<br>
-# CHECK: include_directories[  0] = 'Directory5a'<br>
-# CHECK: include_directories[  1] = 'Directory5b'<br>
+# CHECK: include_directories[  0] = .debug_str[0x00000045] = "Directory5a"<br>
+# CHECK: include_directories[  1] = .debug_str[0x00000051] = "Directory5b"<br>
 # CHECK-NOT: include_directories<br>
----------------<br>
dblaikie wrote:<br>
> Should this print as ".debug_line_str"?<br>
No, this is correct.  The referenced string section depends on the form, not on the section where the reference is made.<br>
This test uses FORM_strp in the directory table and FORM_line_strp in the file table, to verify correct printing for both cases.  In practice I wouldn't expect a producer to mix-n-match like this, but for a test it seemed like a good idea.<br></blockquote><div><br>Fair enough - might be worth a comment in a test.<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/D42802" rel="noreferrer" target="_blank">https://reviews.llvm.org/D42802</a><br>
<br>
<br>
<br>
</blockquote></div></div>