[PATCH] D46966: [lld] Use a real timestamp, with the option to use hash or explicit value

[Zachary Turner via llvm-commits]

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?
On Wed, May 16, 2018 at 11:45 AM Peter Collingbourne via Phabricator <
reviews at reviews.llvm.org> wrote:

> pcc added inline comments.
>
>
> ================
> Comment at: lld/COFF/Driver.cpp:1038
> +  if (auto *Arg = Args.getLastArg(OPT_timestamp)) {
> +    if (Arg.equals_lower("hash")) {
> +      Config->Timestamp = 0;
> ----------------
> It looks like this is spelt `/Brepro` in the MSVC linker. Should we do the
> same?
>
>
> https://reviews.llvm.org/D46966
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180516/2fb8000f/attachment.html>