Re: r232721 - Add option to switch off putting header modules into the dependency file.

2015-08-11 Thread Richard Smith via cfe-commits
On Tue, Aug 11, 2015 at 11:49 AM, Manuel Klimek wrote: > Back in the day I'm pretty sure you agreed with the general concept :) > Yeah, I don't think I fully understood the reason for having a cc1 -module-file-deps flag at the time. http://reviews.llvm.org/D8378 > I remember that we had problem

Re: r232721 - Add option to switch off putting header modules into the dependency file.

2015-08-11 Thread Manuel Klimek via cfe-commits
Back in the day I'm pretty sure you agreed with the general concept :) http://reviews.llvm.org/D8378 I remember that we had problems with absolute paths ending up in the .d file (which should not happen if none of the paths provided are absolute), but I don't remember why we went for not writing th

Fwd: r232721 - Add option to switch off putting header modules into the dependency file.

2015-08-11 Thread Richard Smith via cfe-commits
On Thu, Mar 19, 2015 at 5:00 AM, Manuel Klimek wrote: > Author: klimek > Date: Thu Mar 19 07:00:22 2015 > New Revision: 232721 > > URL: http://llvm.org/viewvc/llvm-project?rev=232721&view=rev > Log: > Add option to switch off putting header modules into the dependency file. > Why? It is not corr