[PATCH] D23934: Add a -ffixed-date-time= flag that sets the initial value of __DATE__, __TIME__, __TIMESTAMP__

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 29 06:32:10 PST 2016


hfinkel added a comment.

In https://reviews.llvm.org/D23934#632216, @emaste wrote:

> Please also accept `SOURCE_DATE_EPOCH` set in the environment -- see https://reproducible-builds.org/specs/source-date-epoch/


It looks like there's reasonable adoption of this: https://wiki.debian.org/ReproducibleBuilds/TimestampsProposal#Reading_the_variable and gcc >= 7 will support it.

> Also although I'm generally leery of options auto-detecting the argument format, I think we should be able to pass an epoch timestamp to -ffixed-date-time.

I'd also find this convenient. The auto-detection should be unambiguous (if it is an integer, then it is an epoch).


https://reviews.llvm.org/D23934





More information about the cfe-commits mailing list