[llvm-bugs] [Bug 34608] New: LLD fails to build with an installed LLVM due to unusable llvm/WindowsManifest/WindowsManifestMerger.h

[via llvm-bugs]

https://bugs.llvm.org/show_bug.cgi?id=34608

            Bug ID: 34608
           Summary: LLD fails to build with an installed LLVM due to
                    unusable llvm/WindowsManifest/WindowsManifestMerger.h
           Product: lld
           Version: unspecified
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: release blocker
          Priority: P
         Component: COFF
          Assignee: unassignedbugs at nondot.org
          Reporter: orivej at gmx.fr
                CC: llvm-bugs at lists.llvm.org

https://reviews.llvm.org/D36255 has added

     #include "llvm/WindowsManifest/WindowsManifestMerger.h"

to COFF/DriverUtils.cpp.  WindowsManifestMerger.h has

     #include "llvm/Config/config.h"

which is a private llvm header: it is not installed when llvm is installed. 
This causes lld build to fail:

     In file included from ../COFF/DriverUtils.cpp:35:0:
     /usr/include/llvm/WindowsManifest/WindowsManifestMerger.h:29:32: fatal
error: llvm/Config/config.h: No such file or directory

Since WindowsManifestMerger.h is a public header (it is installed with llvm and
may be included by lld), it should not include llvm/Config/config.h. (It may
include llvm/Config/llvm-config.h.)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20170914/7154b6c4/attachment.html>