[PATCH] [PECOFF] Support a new type of weak symbol.
kledzik at apple.com
kledzik at apple.com
Thu Mar 6 17:08:53 PST 2014
================
Comment at: include/lld/Core/DefinedAtom.h:102
@@ -102,1 +101,3 @@
+ mergeByContent, // Merge with other constants with same content
+ mergeSameSize, // Another atom with different size is error
};
----------------
Rui Ueyama wrote:
> kledzik at apple.com wrote:
> > Maybe mergeSameNameAndSize.
> "SameName" is implied for all merge options, so I guess it's extraneous.
"name" is only implied for the "weak" cases. The mergeByContent case does not use the name
http://llvm-reviews.chandlerc.com/D2996
BRANCH
master
ARCANIST PROJECT
lld
More information about the llvm-commits
mailing list