Re: [PATCH] D37954: Expand absolute system header paths when using -MD depfiles

2017-09-17 Thread Peter Wu via cfe-commits
Hi Nico, On Sun, Sep 17, 2017 at 10:47:24AM -0400, Nico Weber wrote: > Have you checked how much the additional stat()ing added in this patch > slows down builds? Additional cost: zero :-) The information is already collected by FileManager::getFile(), calling tryGetRealPathName just reads infor

Re: [PATCH] D37954: Expand absolute system header paths when using -MD depfiles

2017-09-17 Thread Nico Weber via cfe-commits
Have you checked how much the additional stat()ing added in this patch slows down builds? On Sep 16, 2017 11:49 PM, "Peter Wu via Phabricator via cfe-commits" < cfe-commits@lists.llvm.org> wrote: > Lekensteyn added a comment. > > I tried to contact Simon (the author of the GCC) patch with a quest