<div dir="ltr"><div dir="ltr">On Wed, Feb 20, 2019 at 5:07 PM Nico Weber via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">thakis added a comment.<br>
<br>
> Hmm, that's right. That's probably a good reason why we can't make P the default behavior.<br>
<br>
Doesn't that make the output dependent on the build directory path? In Chromium, we're very careful to try and make the build independent of the build directory name for deterministic builds. I think changing the default here is bad for this reason. If you have your mind set on this however, is there an opt out flag? (We don't use llvm-ar in chromium at the moment so this doesn't affect us, but it's maybe a concern you didn't think about, so I figured I'd mention it.)<br></blockquote><div><br></div><div>I'm not sure if I follow. You don't want to make P default? If so, that's I think the conclusion of this thread. P is not enabled by default (unless you specify P).</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D57927/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D57927/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D57927" rel="noreferrer" target="_blank">https://reviews.llvm.org/D57927</a><br>
<br>
<br>
<br>
</blockquote></div></div>