[cfe-commits] [Request for approval] header search (2)

[Rafael Espindola]

On 12 May 2010 17:18, mike-m <mikem.llvm at gmail.com> wrote:
> DESCRIPTION:
>  Moved clang-builtin include dir position to immediately precede C includes.
>  This aligns with how gcc compiler does things. Fixes #include_next
>  shenanigans that lead to failures when clang++ is used to compile source
>  with <cstddef>. Specifically, libstdc++ versions 4.3.[012] is at issue,
>  which ships bundled on Debian 5.04, Ubuntu 8.10 and Fedora 10.
>
>  Example error from Ubuntu 8.10:
>
>  > /usr/include/c++/4.3/cstddef:47:15: fatal error: 'stddef.h' file not found
>  > #include_next <stddef.h>
>  >               ^
>  > 1 error generated.

In the ubuntu system I have the include order the only difference to
gcc (and g++) is that the gcc dirs

 /usr/lib/gcc/x86_64-linux-gnu/4.4.1/include
 /usr/lib/gcc/x86_64-linux-gnu/4.4.1/include-fixed
 /usr/include/x86_64-linux-gnu

are replaced with the clang dir

 /home/espindola/llvm/llvm-build/Release/lib/clang/2.0/include

Unfortunately I don't have a windows machine to test this, but
otherwise looks good.

Thanks!
-- 
Rafael Ávila de Espíndola