From 4ab50b858ed7c3dfee8cef040b5ed43c863e44d7 Mon Sep 17 00:00:00 2001 From: Reid Kleckner Date: Tue, 18 Dec 2018 01:36:06 +0000 Subject: [codeview] Update comment on aligning symbol records llvm-svn: 349433 --- llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp') diff --git a/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp b/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp index 78e5f5d2..4132b1a 100644 --- a/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp +++ b/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp @@ -2831,8 +2831,10 @@ MCSymbol *CodeViewDebug::beginSymbolRecord(SymbolKind SymKind) { } void CodeViewDebug::endSymbolRecord(MCSymbol *SymEnd) { - // Symbol records in object files are not aligned, although we are considering - // it for linker performance reasons. + // MSVC does not pad out symbol records to four bytes, but LLVM does to avoid + // an extra copy of every symbol record in LLD. This increases object file + // size by less than 1% in the clang build, and is compatible with the Visual + // C++ linker. OS.EmitValueToAlignment(4); OS.EmitLabel(SymEnd); } -- cgit v1.1