We can do /Brepro but we do a little more than /Brepro.  We allow /timestamp:hash as well as setting an explicit value.  We could keep it for compatibility though and map it to /timestamp:0.  Thoughts?<br><div class="gmail_quote"><div dir="ltr">On Wed, May 16, 2018 at 11:45 AM Peter Collingbourne via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">pcc added inline comments.<br>
<br>
<br>
================<br>
Comment at: lld/COFF/Driver.cpp:1038<br>
+  if (auto *Arg = Args.getLastArg(OPT_timestamp)) {<br>
+    if (Arg.equals_lower("hash")) {<br>
+      Config->Timestamp = 0;<br>
----------------<br>
It looks like this is spelt `/Brepro` in the MSVC linker. Should we do the same?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D46966" rel="noreferrer" target="_blank">https://reviews.llvm.org/D46966</a><br>
<br>
<br>
<br>
</blockquote></div>