<div dir="ltr">Usually we try to reduce the source code before generating the IR test case (looks like this IR may've been hand reduced further? But I may be mistaken) for debug info tests & include the source in a comment at the top of the test so it's easy to understand the basic intent of the test case - since reconstructing the intent/semantics from the metadata is somewhat non-trivial.<br><br>(also, sometimes we checkin binaries to test the dumper - we don't always, and I'm not suggesting it here, just mentioning it as a thought - sometimes we just treat the dumper as "golden" and assume we're just testing the LLVM IR/CodeGen behavior)<br><br>- Dave</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 26, 2016 at 8:28 AM, Dehao Chen via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">danielcdh updated this revision to Diff 55008.<br>
danielcdh added a comment.<br>
<br>
Add comment and reformat.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D19522" rel="noreferrer" target="_blank">http://reviews.llvm.org/D19522</a><br>
<br>
Files:<br>
  lib/DebugInfo/DWARF/DWARFDebugLine.cpp<br>
  test/DebugInfo/X86/discriminator2.ll<br>
<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">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>
<br></blockquote></div><br></div>