[PATCH] D46966: [lld] Use a real timestamp, with the option to use hash or explicit value

[Zachary Turner via Phabricator via llvm-commits]

zturner added inline comments.


================
Comment at: lld/COFF/Driver.cpp:1041
+      Config->TimestampIsHash = true;
+      return;
+    } else {
----------------
ruiu wrote:
> This seems like a mistake. If your test passed with this patch, you should add a test to execute this path.
Actually my test didn't pass.  I just didn't re-run them after I inlined the function.  Thanks!


https://reviews.llvm.org/D46966