<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Fri, Dec 1, 2017 at 3:50 PM Jim Ingham 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">jingham added a comment.<br>
<br>
Cool.  We do need to make sure people don't start writing tests against it yet, however.  That would be wasted effort.<br></blockquote><div><br></div><div>I don't think it follows that it would be a wasted effort.  Anything that increases test coverage today is the exact opposite of a wasted effort.  Currently it prints useful information about symbols.  The format may not have been designed for writing tests against, but it is not that bad either.  Symbols, sections, and headers are printed on individual lines</div><div><br></div><div>I don't think we should build up a suite of several hundred debug info tests around this particular format, but I don't think we should hesitate to add some when an opportunity presents itself either.  Often what the best format actually is only becomes evident through experience (i.e. iterations).</div><div><br></div><div>A first idea might be to understand why each type shows up 3 times in the output.  E.g.</div><div><br></div><div><span style="color:rgb(33,33,33);font-size:13px">0BA597D8:   Type{0x0000005f} , compiler_type = 0x0ba34d70 signed char **</span><br style="color:rgb(33,33,33);font-size:13px"><span style="color:rgb(33,33,33);font-size:13px">0BA597D8:   Type{0x0000005f} , compiler_type = 0x0ba34d70 signed char **</span>  <br></div><div>...</div><div><div><span style="color:rgb(33,33,33);font-size:13px">0BA597D8:   Type{0x0000005f} , compiler_type = 0x0ba34d70 signed char **</span>  <br></div><br class="inbox-inbox-Apple-interchange-newline"></div><div>could be just a bug in the dump function, but another possibility is that we are actually storing the same thing 3 times.  If that is the case, then after fixing it, a test could then verify that a line that currently prints 3 times only prints once after a fix.</div></div></div>