[cfe-dev] [patch] Add configure option for C include search path

Chandler Carruth chandlerc at google.com
Sat Nov 14 16:12:14 PST 2009


On Sat, Nov 14, 2009 at 2:51 PM, Rafael Espindola <espindola at google.com> wrote:
>> +1, sounds like a great compromise to me, and meshes well w/ the idea
>> of inserting a single custom "target" in the future distinct from any
>> of the builtin configurations.
>
> The attached patches implement the proposed option. Are they OK? I
> tested that they build with cmake too :-)

Do we want to default the C++ include arch to the target arch? It
still won't be used unless an explicit root is given.

Also, spelling nit:
--- a/include/llvm/Config/config.h.cmake
+++ b/include/llvm/Config/config.h.cmake
@@ -12,6 +12,18 @@
 /* Directories clang will search for headers */
 #define C_INCLUDE_DIRS "${C_INCLUDE_DIRS}"

+/* Directorie clang will search for libstdc++ headers */
s/Directorie/Directory/

>
>> -Chandler
>
> Cheers,
> --
> Rafael Ávila de Espíndola
>




More information about the cfe-dev mailing list