[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