[PATCH] D36304: [lld] [COFF] Add support for aligncomm directives

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 4 01:58:19 PDT 2017


mstorsjo created this revision.

These are emitted for comm symbols in object files, when targeting a GNU environment.

Alternatively, just ignore them since we already align CommonChunk to the natural size of the content (up to 32 bytes). That would only trade away the possibility to overalign small symbols, which doesn't sound like something that might not need to be handled?


https://reviews.llvm.org/D36304

Files:
  COFF/Chunks.cpp
  COFF/Chunks.h
  COFF/Config.h
  COFF/Driver.cpp
  COFF/Driver.h
  COFF/DriverUtils.cpp
  COFF/Options.td
  COFF/Symbols.h
  test/COFF/common-alignment.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36304.109692.patch
Type: text/x-patch
Size: 7393 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170804/8f60781e/attachment.bin>


More information about the llvm-commits mailing list