[PATCH] D69221: [clang][darwin] Fix search path logic for C_INCLUDE_DIRS

2019-10-19 Thread Marco Hinz via Phabricator via cfe-commits
mhinz created this revision. mhinz added reviewers: ldionne, rsmith. mhinz added a project: clang. Herald added subscribers: cfe-commits, dexonsmith. For each absolute path given to C_INCLUDE_DIRS, we want it to be added as-is to the include search path. Relative paths should be prefixed with the

[PATCH] D69221: [clang][darwin] Fix search path logic for C_INCLUDE_DIRS

2019-11-27 Thread Marco Hinz via Phabricator via cfe-commits
mhinz updated this revision to Diff 231266. mhinz added a comment. Herald added subscribers: aheejin, jgravelle-google, sbc100, dschuff. Apply fix to all toolchains. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69221/new/ https://reviews.llvm.org/D69221 Files: clang/lib/Driver/ToolC

[PATCH] D69221: [clang][darwin] Fix search path logic for C_INCLUDE_DIRS

2019-11-27 Thread Marco Hinz via Phabricator via cfe-commits
mhinz added a comment. I applied the fix to all toolchains now, but I'm unsure how to write a test for it. Neither could I find tests for similar options like `-DDEFAULT_SYSROOT`. My use case: I want to build a clang that has the same search path as the clang shipped with macOS. $ /usr/bin/c

[PATCH] D69221: [clang][darwin] Fix search path logic for C_INCLUDE_DIRS

2020-01-30 Thread Marco Hinz via Phabricator via cfe-commits
mhinz added a comment. In D69221#1848425 , @ldionne wrote: > Do you have commit access? No. :) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D69221/new/ https://reviews.llvm.org/D69221 ___ cfe-comm