pgousseau added a comment.

In http://reviews.llvm.org/D20867#482465, @rsmith wrote:

> You have two independent functional changes in this patch: one adds a flag to 
> control the emission of timestamps into PCH files, and the other re-enables 
> timestamp checking on Win32. Please separate them out into distinct patches 
> to be committed separately.
>
> Both parts of this LGTM.


Splitting the patch sounds good, thanks Richard and Bruno for reviewing, will  
push soon.


http://reviews.llvm.org/D20867



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to