<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Dec 10, 2016 2:12 PM, "Rafael Avila de Espindola" <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">Rui Ueyama via Phabricator via llvm-commits<br>
<br>
<br>
</div>> +  // If threading is disabled, use the default string table builder<br>
> +  // because our concurrent one is not as fast as the single-threaded<br>
<div class="quoted-text">> +  // one on single core.<br>
> +  if (!Config->Threads || StringRef(getenv("LLD_TEST")) == "1") {<br>
> +    Builder.reset(new StringTableBuilder(<wbr>StringTableBuilder::RAW, Alignment));<br>
</div><div class="quoted-text">>      finalizeNoTailMerge();<br>
> +    return;<br>
> +  }<br>
> +<br>
<br>
</div>This means we produce different results with threads disabled?<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Yes. But as long as you pass the same command line arguments, results are the same.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What is the LLD_TEST thing?<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">That's for debugging. Removed.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div></div>