[llvm-commits] [cfe-commits] [PATCH] autoconf: add private config.h to clang

Eric Christopher echristo at apple.com
Tue Jan 31 13:34:05 PST 2012


On Jan 30, 2012, at 1:01 PM, nobled wrote:

> This already exists in the CMake build, which is part of what makes
> building clang separately from llvm via cmake possible. This cleans up
> that discrepancy between the build systems.
> 
> I'll just add the minimal file include/clang/Config/config.h.in (with
> the same contents as its config.h.cmake counterpart), then add it to
> LLVM's configure script so it gets generated, then remove the #ifdef
> logic from clang that was conditionally including the cmake generated
> header.
> 
> Okay to commit?

Seems reasonable. Didn't think we'd need one, but it's fine with me if we want one.

-eric



More information about the llvm-commits mailing list