Re: r269769 - [PCH] Fixed bug with preamble invalidation when overridden files change

2016-05-23 Thread Bruno Cardoso Lopes via cfe-commits
Ping on the testcase :-) On Tue, May 17, 2016 at 9:36 AM, Nico Weber via cfe-commits wrote: > Is it possible to write a test for this? > > On Tue, May 17, 2016 at 10:34 AM, Cameron Desrochers via cfe-commits > wrote: >> >> Author: cameron314 >> Date: Tue May 17 09:34:53 2016 >> New Revision: 269

Re: r269769 - [PCH] Fixed bug with preamble invalidation when overridden files change

2016-05-17 Thread Nico Weber via cfe-commits
Is it possible to write a test for this? On Tue, May 17, 2016 at 10:34 AM, Cameron Desrochers via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: cameron314 > Date: Tue May 17 09:34:53 2016 > New Revision: 269769 > > URL: http://llvm.org/viewvc/llvm-project?rev=269769&view=rev > Log: >

r269769 - [PCH] Fixed bug with preamble invalidation when overridden files change

2016-05-17 Thread Cameron Desrochers via cfe-commits
Author: cameron314 Date: Tue May 17 09:34:53 2016 New Revision: 269769 URL: http://llvm.org/viewvc/llvm-project?rev=269769&view=rev Log: [PCH] Fixed bug with preamble invalidation when overridden files change When remapped files were changed, they would not always cause the preamble's PCH to be