[PATCH] D37863: Fix bug 34608 by moving private header out of public header.WindowsManifestMerger.h should not include llvm/Config/config.h, since it is private. The include has been moved to the source instead.

[Orivej Desh via Phabricator via llvm-commits]

orivej added inline comments.


================
Comment at: llvm/lib/WindowsManifest/WindowsManifestMerger.cpp:16
 #include "llvm/Support/MemoryBuffer.h"
+#include "llvm/WindowsManifest/WindowsManifestMerger.h"
 
----------------
Shouldn't the synonymous header be always the first include (to check that it is self-sufficient)?


https://reviews.llvm.org/D37863