<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 25, 2015 at 10:49 AM, Paul Robinson <span dir="ltr"><<a href="mailto:Paul_Robinson@playstation.sony.com" target="_blank">Paul_Robinson@playstation.sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">REPOSITORY<br>
rL LLVM<br>
<br>
================<br>
Comment at: test/DebugInfo/X86/missing-file-line.ll:3<br>
@@ +2,3 @@<br>
+<br>
+; RUN: llc -mtriple=x86_64-linux-gnu -filetype=obj %s -o - | llvm-dwarfdump -debug-dump=all - > %t<br>
+; RUN: FileCheck --check-prefix=CHECK1 %s < %t<br>
----------------<br>
</span><span class="">pcc wrote:<br>
> dblaikie wrote:<br>
> > Ideally just pipe dwarfdump's output straight to FileCheck without the intermediate %t - then just use a single set of CHECK (rather than 3 separate CHECK prefixes) lines for the tests.<br>
> Previously I used `CHECK-DAG`, but I couldn't see a way to do the `CHECK-NOT` checks within a single run of FileCheck while keeping them order independent.<br>
</span>That would hard-code an order for the three DIEs, which is probably okay for the structure/member but I'm not so sure about the typedef.<br></blockquote><div><br>Yeah, this is a known limitation of the way we write tests, but we tend to prefer to still do it this way - maybe one day we'll be able to beef up FileCheck (or make llvm-dwarfdump smart enough to have a sort option or something) to make this testing more resilient. For now I'd rather just be consistent rather than do anything too fancy. The ordering doesn't change too often.<br><br>- David<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D8588" target="_blank">http://reviews.llvm.org/D8588</a><br>
<br>
EMAIL PREFERENCES<br>
<a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>