<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 10, 2016 at 9:55 AM, Xinliang David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I meant the op1 part can be simplified -- but not necessariliy need to.<br></blockquote><div><br></div><div>Ok, I think I prefer it the way it is now, checking against the expected MD5 hash values. Will leave it as-is.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
David<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Feb 10, 2016 at 9:53 AM, Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> wrote:<br>
> tejohnson added inline comments.<br>
><br>
> ================<br>
> Comment at: test/tools/gold/X86/thinlto.ll:27-31<br>
> @@ -26,5 +26,7 @@<br>
>  ; COMBINED-NEXT: <VALUE_SYMTAB<br>
> -; COMBINED-NEXT: <COMBINED_FNENTRY {{.*}} record string = '{{f|g}}'<br>
> -; COMBINED-NEXT: <COMBINED_FNENTRY {{.*}} record string = '{{f|g}}'<br>
> +; Check that the format is: op0=offset, op1=funcguid, where funcguid is<br>
> +; the lower 64 bits of the function name MD5.<br>
> +; COMBINED-NEXT: <COMBINED_FNENTRY abbrevid={{[0-9]+}} op0={{[0-9]+}} op1={{-3706093650706652785|-5300342847281564238}}<br>
> +; COMBINED-NEXT: <COMBINED_FNENTRY abbrevid={{[0-9]+}} op0={{[0-9]+}} op1={{-3706093650706652785|-5300342847281564238}}<br>
>  ; COMBINED-NEXT: </VALUE_SYMTAB<br>
><br>
> ----------------<br>
> I think that's what I have here already? I could shorten this a bit by replacing the intermediate parts with {{.*}}. E.g.:<br>
><br>
> ; COMBINED-NEXT: <COMBINED_FNENTRY {{.*}} {{-3706093650706652785|-5300342847281564238}}<br>
><br>
> Is that preferable?<br>
><br>
><br>
> <a href="http://reviews.llvm.org/D17028" rel="noreferrer" target="_blank">http://reviews.llvm.org/D17028</a><br>
><br>
><br>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>