[LLVMbugs] [Bug 7809] New: llvm/Config/config.h pollutes user namespace

bugzilla-daemon at llvm.org bugzilla-daemon at llvm.org
Wed Aug 4 08:22:39 PDT 2010


           Summary: llvm/Config/config.h pollutes user namespace
           Product: libraries
           Version: trunk
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P
         Component: Target Description Classes
        AssignedTo: unassignedbugs at nondot.org
        ReportedBy: jdenny at etinternational.com
                CC: llvmbugs at cs.uiuc.edu

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.

Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

More information about the llvm-bugs mailing list