[PATCH] Fix merge-by-content to not merge atoms with different custom sections

kledzik at apple.com kledzik at apple.com
Wed Oct 1 18:39:23 PDT 2014


Hi ruiu, t.p.northover, shankar.easwaran,

The mergeByContent attribute on atoms triggers the symbol table to coalesce atoms with the exact same content.  The problem is that atoms can also have a required custom section.  The coalescing should never change the custom section of an atom.  

The fix is to only consider to atoms to have the same content if their sectionChoice() and customSectionName() attributes match.

http://reviews.llvm.org/D5570

Files:
  lib/Core/SymbolTable.cpp
  test/core/custom-section-coalesce.objtxt
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D5570.14305.patch
Type: text/x-patch
Size: 3259 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141002/9e05bc8f/attachment.bin>


More information about the llvm-commits mailing list