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

Joerg Sonnenberger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 17 16:17:11 PDT 2017


joerg added a comment.

ninja is not the only consumer of this. For human inspection, it can be quite surprising to see symlinks resolved, i.e. /usr/include/machine on NetBSD. Build systems have different ideas on whether they want absolute resolved paths or not, so it seems like ninja should be able to cope with either format. This is a lossy transformation, so I'm somewhat reluctant to agree with it.


https://reviews.llvm.org/D37954





More information about the cfe-commits mailing list