[LLVMdev] llvm/Config/config.h namespace problem

Joel E. Denny jdenny at etinternational.com
Tue Aug 3 10:05:41 PDT 2010


The LLVM headers llvm-c/Target.h and llvm/Target/TargetSelect.h include 
the header llvm/Config/config.h, which defines common macros, such as 
PACKAGE_BUGREPORT.  If a project that uses LLVM includes any of these 
headers, the project's own macros of the same names can be redefined.

Most advice I have seen says that installed headers should never define 
such common macros.  Thus, an internal config.h should never be installed:


Would you please consider replacing the current llvm/Config/config.h with 
a trimmed down version that contains only the definitions that must be 
present in LLVM's installed headers?  PACKAGE_* macros certainly do not 
need to be there.


More information about the llvm-dev mailing list