The checksums had already been placed in the IR, this patch allows
MCCodeView to actually write it out to an MCStreamer.
Move private config.h header dependency out of public header file.
Addresses Bug 34608
Paths
| Differential 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. ClosedPublic Authored by ecbeckmann on Sep 14 2017, 1:01 PM.
Details Summary The checksums had already been placed in the IR, this patch allows Move private config.h header dependency out of public header file. Addresses Bug 34608
Diff Detail
Event Timelineecbeckmann retitled this revision from 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. to 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..Sep 14 2017, 1:02 PM ecbeckmann changed the visibility from "Public (No Login Required)" to "ecbeckmann (Eric Beckmann)". ecbeckmann changed the visibility from "ecbeckmann (Eric Beckmann)" to "Public (No Login Required)".Sep 14 2017, 1:06 PM This revision is now accepted and ready to land.Sep 14 2017, 1:25 PM
ecbeckmann added inline comments.
Closed by commit rL313312: Fix bug 34608 by moving private header out of public header. (authored by ecbeckmann). · Explain WhySep 14 2017, 4:02 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 115267 clang/lib/CodeGen/CGDebugInfo.cpp
llvm/include/llvm/IR/DIBuilder.h
llvm/include/llvm/IR/DebugInfoMetadata.h
llvm/include/llvm/MC/MCCodeView.h
llvm/include/llvm/MC/MCObjectStreamer.h
llvm/include/llvm/MC/MCStreamer.h
llvm/include/llvm/WindowsManifest/WindowsManifestMerger.h
llvm/lib/AsmParser/LLParser.cpp
llvm/lib/Bitcode/Reader/MetadataLoader.cpp
llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
llvm/lib/DebugInfo/CodeView/DebugChecksumsSubsection.cpp
llvm/lib/IR/AsmWriter.cpp
llvm/lib/IR/DebugInfoMetadata.cpp
llvm/lib/IR/Verifier.cpp
llvm/lib/MC/MCAsmStreamer.cpp
llvm/lib/MC/MCCodeView.cpp
llvm/lib/MC/MCObjectStreamer.cpp
llvm/lib/MC/MCParser/AsmParser.cpp
llvm/lib/MC/MCStreamer.cpp
llvm/lib/WindowsManifest/WindowsManifestMerger.cpp
llvm/test/DebugInfo/COFF/asm.ll
llvm/test/DebugInfo/COFF/inlining.ll
llvm/test/DebugInfo/COFF/multifile.ll
llvm/test/DebugInfo/COFF/simple.ll
llvm/test/MC/AArch64/coff-debug.ll
llvm/test/MC/COFF/cv-loc.s
llvm/tools/llvm-readobj/COFFDumper.cpp
llvm/unittests/IR/MetadataTest.cpp
|
Shouldn't the synonymous header be always the first include (to check that it is self-sufficient)?