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

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


Hi,

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:

  http://stackoverflow.com/questions/1810216/autoconf-where-does-config-h-go
  http://lists.gnu.org/archive/html/autoconf/2004-09/msg00151.html

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.

Thanks.



More information about the llvm-dev mailing list