<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 1, 2016 at 8:33 AM, pierre gousseau via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">pgousseau created this revision.<br>
pgousseau added reviewers: rsmith, thakis.<br>
pgousseau added subscribers: cfe-commits, wristow, probinson, gbedwell, bruno, cameron314.<br>
<br>
On Linux, if the timestamp of a header file, included in the pch, is modified, then including the pch without regenerating it causes a fatal error, which is reasonable.<br>
On Windows the check is ifdefed out, allowing the compilation to continue in a broken state.<br>
The root of the broken state is that, if timestamps dont match, the preprocessor will reparse a header without discarding the pch data.<br>
This leads to "#pragma once" header to be included twice.<br>
The reason behind the ifdefing of the check lacks documentation, and was done 6 years ago.<br></blockquote><div><br></div><div>It's documented in the comment you deleted:</div><div><br></div><div><div>       // In our regression testing, the Windows file system seems to</div><div>       // have inconsistent modification times that sometimes<br></div></div><div>       // erroneously trigger this error-handling path.<br></div><div><br></div><div>We should confirm whether this is in fact still the case. Maybe this is caused by building on a networked file system, where a locally-changed file might have a different mtime locally and remotely (the local mtime may be precise where the remote one has been rounded to a multiple of 2 seconds by an underlying FAT filesystem)? If it's something like that, we could perhaps work around this by rounding the mtime to a multiple of 2 seconds ourselves.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
This change tentatively removes the ifdefing and adds a cc1 option to disable the inclusion of timestamps in pch files, giving some flexibility to build systems such as distributed builds.<br>
<br>
This change is a follow up to the discussion started in <a href="http://reviews.llvm.org/D20243" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20243</a><br>
<br>
<a href="http://reviews.llvm.org/D20867" rel="noreferrer" target="_blank">http://reviews.llvm.org/D20867</a><br>
<br>
Files:<br>
  include/clang/Driver/CC1Options.td<br>
  include/clang/Frontend/FrontendOptions.h<br>
  lib/Frontend/CompilerInvocation.cpp<br>
  lib/Frontend/FrontendActions.cpp<br>
  lib/Serialization/ASTReader.cpp<br>
  test/PCH/Inputs/include-timestamp-pch.h<br>
  test/PCH/Inputs/include-timestamp.h<br>
  test/PCH/include-timestamp.cpp<br>
<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div></div>