[cfe-commits] [PATCH] Add the resource directory to the search path for Driver::GetFilePath

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Sat Sep 3 20:33:42 PDT 2011


On 11-09-01 5:16 PM, Peter Collingbourne wrote:
> Hi,
>
> This patch adds the resource directory to the search path for
> Driver::GetFilePath, as well as the search path printed by
> -print-search-dirs.
>
> The main purpose of this change is to cause -print-file-name=include
> to print the path to the include directory under Clang's resource
> directory, instead of the system compiler's include directory, whose
> header files Clang may not be able to parse.  Some build scripts will
> do something like:
>    $(CC) -nostdinc -I`$(CC) -print-file-name=include`
> to exclude all header paths except the compiler's (I've observed this
> happening with uclibc).
>
> OK to commit?

I think it is OK. I know it is hard to test the driver, but for this 
patch we might be able to check that there is as "/clang/" in the output 
of -print-file-name=include, no?

OK with the test.

> Thanks,
>
>

Cheers,
Rafael



More information about the cfe-commits mailing list