[PATCH] D135045: [Frontend] Recognize environment variable SOURCE_DATE_EPOCH

[Fangrui Song via Phabricator via cfe-commits]

MaskRay marked an inline comment as done.
MaskRay added inline comments.


================
Comment at: clang/lib/Lex/PPMacroExpansion.cpp:1612
+      time_t TT = *getPreprocessorOpts().SourceDateEpoch;
+      std::tm *TM = std::gmtime(&TT);
       Result = asctime(TM);
----------------
ychen wrote:
> To be consistent with existing code, it might be better to use `std::localtime` instead of `std::gmtime`?
No. This must use `std::gmtime`. localtime would introduce an undesired UTC time offset.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135045/new/

https://reviews.llvm.org/D135045