[PATCH] D63116: Let writeWindowsResourceCOFF() take a TimeStamp parameter

[Nico Weber via Phabricator via llvm-commits]

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


================
Comment at: llvm/trunk/tools/llvm-cvtres/llvm-cvtres.cpp:94
+  if (Now < 0 || !isUInt<32>(Now))
+    return UINT32_MAX;
+  return static_cast<uint32_t>(Now);
----------------
ruiu wrote:
> Can this happen? Is this for the year 2038 problem?
I just moved this function around so I'm not sure, but that was my guess as well.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63116