<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 19, 2017 at 9:56 AM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I guess non deterministic is the wrong word.  "Volatile" or "fragile" might be better.  Imagine a number like a file size, where when you add something the size changes.  In this case I believe it was a stream length, where adding more data changed the size of the stream, but where someone writing/reading the test could not possibly know what number was supposed to be there, so like an implementation detail <br></blockquote><div><br>Ah, yep yep - no worries (yeah, we do similar things in the DWARF testing for sure - just matching regex to offsets rather than hard coding them, so they're resilient to new attributes being added/removed, etc).<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div dir="ltr">On Mon, Jun 19, 2017 at 9:40 AM David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Nondeterministic?<br><br>Is that a necessary function of the format or what? LLVM tools usually try very hard to be deterministic - I'd be wary of making any intentionally non-deterministic.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 15, 2017 at 3:42 PM Zachary Turner via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">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">Author: zturner<br>
Date: Thu Jun 15 17:41:45 2017<br>
New Revision: 305520<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=305520&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=305520&view=rev</a><br>
Log:<br>
Remove some non-determinstic numbers from pdb test.<br>
<br>
Modified:<br>
    lld/trunk/test/COFF/pdb.test<br>
<br>
Modified: lld/trunk/test/COFF/pdb.test<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/test/COFF/pdb.test?rev=305520&r1=305519&r2=305520&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/test/COFF/pdb.test?rev=305520&r1=305519&r2=305520&view=diff</a><br>
==============================================================================<br>
--- lld/trunk/test/COFF/pdb.test (original)<br>
+++ lld/trunk/test/COFF/pdb.test Thu Jun 15 17:41:45 2017<br>
@@ -183,17 +183,17 @@ RAW-NEXT:         IMAGE_SCN_MEM_READ<br>
 RAW:                             Section Map<br>
 RAW-NEXT: ============================================================<br>
 RAW-NEXT:   Section 0000 | ovl = 0, group = 0, frame = 0, name = 1<br>
-RAW-NEXT:                  class = 65535, offset = 0, size = 12<br>
+RAW-NEXT:                  class = 65535, offset = 0, size =<br>
 RAW-NEXT:                  flags = read | 32 bit addr | selector<br>
 RAW-NEXT:   Section 0001 | ovl = 1, group = 0, frame = 0, name = 2<br>
-RAW-NEXT:                  class = 65535, offset = 0, size = 22<br>
+RAW-NEXT:                  class = 65535, offset = 0, size =<br>
 RAW-NEXT:                  flags = read | execute | 32 bit addr | selector<br>
 RAW-NEXT:   Section 0002 | ovl = 2, group = 0, frame = 0, name = 3<br>
-RAW-NEXT:                  class = 65535, offset = 0, size = 8<br>
+RAW-NEXT:                  class = 65535, offset = 0, size =<br>
 RAW-NEXT:                  flags = read | 32 bit addr | selector<br>
 RAW-NEXT:   Section 0003 | ovl = 3, group = 0, frame = 0, name = 4<br>
-RAW-NEXT:                  class = 65535, offset = 0, size = 119<br>
+RAW-NEXT:                  class = 65535, offset = 0, size =<br>
 RAW-NEXT:                  flags = read | 32 bit addr | selector<br>
 RAW-NEXT:   Section 0004 | ovl = 4, group = 0, frame = 0, name = 5<br>
-RAW-NEXT:                  class = 65535, offset = 0, size = 4294967295<br>
+RAW-NEXT:                  class = 65535, offset = 0, size =<br>
 RAW-NEXT:                  flags = 32 bit addr | absolute addr<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>
</blockquote></div>
</blockquote></div></div>