[cfe-commits] r86956 - /cfe/trunk/lib/Frontend/InitHeaderSearch.cpp

[Nuno Lopes]

> Author: rafael
> Date: Wed Nov 11 23:48:41 2009
> New Revision: 86956
>
> URL: http://llvm.org/viewvc/llvm-project?rev=86956&view=rev
> Log:
> Use the --with-c-include-dirs configure option.
>
> ==============================================================================
> --- cfe/trunk/lib/Frontend/InitHeaderSearch.cpp (original)
> +++ cfe/trunk/lib/Frontend/InitHeaderSearch.cpp Wed Nov 11 23:48:41 2009
> @@ -18,6 +18,7 @@
> #include "clang/Lex/HeaderSearch.h"
> #include "llvm/ADT/SmallString.h"
> #include "llvm/ADT/SmallPtrSet.h"
> +#include "llvm/ADT/StringExtras.h"
> #include "llvm/ADT/Triple.h"
> #include "llvm/Support/raw_ostream.h"
> #include "llvm/System/Path.h"
> @@ -302,6 +303,14 @@
>
> void InitHeaderSearch::AddDefaultCIncludePaths(const llvm::Triple &triple) 
> {
>   // FIXME: temporary hack: hard-coded paths.
> +  if (C_INCLUDE_DIRS != "") {

You may want to write 'if (!*C_INCLUDE_DIRS)' here, as your code seems to be 
a bit dangerous.
Nuno