[PATCH] D19892: Remove LLVM_ENABLE_TIMESTAMPS

[Chris Bieneman via llvm-commits]

beanz created this revision.
beanz added reviewers: bogner, silvas, rnk.
beanz added a subscriber: llvm-commits.

As per the discussion on LLVM-dev this patch proposes removing LLVM_ENABLE_TIMESTAMPS.

The only complicated bit of this patch is the Windows support. On windows we used to log an error if /INCREMENTAL was passed to the linker when timestamps were disabled.

With this change since timestamps in code are always disabled we will always compile on windows with /Brepro unless /INCREMENTAL is specified, and we will log a warning when /INCREMENTAL is specified to notify the user that the build will be non-deterministic.

See: http://lists.llvm.org/pipermail/llvm-dev/2016-May/098990.html

http://reviews.llvm.org/D19892

Files:
  CMakeLists.txt
  cmake/modules/HandleLLVMOptions.cmake
  include/llvm/Config/config.h.cmake
  lib/Support/CommandLine.cpp
  utils/release/test-release.sh

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19892.56074.patch
Type: text/x-patch
Size: 4861 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160503/506e8161/attachment.bin>