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