[PATCH] D53913: [llvm-objcopy] Support --{enable, disable}-deterministic-archives

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 30 17:13:44 PDT 2018


rupprecht added a comment.

In https://reviews.llvm.org/D53913#1281601, @ruiu wrote:

> I feel it should be enabled by default, as deterministic build is a very nice property.


It was before this patch and it still is. This is providing the option to make it not default (if anyone wants that), and also won't break if someone is explicitly running "objcopy --enable-deterministic-archives" and we drop llvm-objcopy in place of that.
I can add a test case for that, actually.


Repository:
  rL LLVM

https://reviews.llvm.org/D53913





More information about the llvm-commits mailing list