[PATCH] [lld][core] support .gnu.linkonce sections

[Shankar Kalpathi Easwaran]

================
Comment at: include/lld/Core/DefinedAtom.h:336
@@ +335,3 @@
+  /// that represents section groups or .gnu.linkonce sections.
+  bool isParentOfAGroup() const {
+    ContentType atomContentType = contentType();
----------------
Rui Ueyama wrote:
> "OfA" sounds a bit odd. I'd name isParentOfGroup or isGroupParent.
Will rename to isGroupParent.

================
Comment at: lib/Core/Resolver.cpp:250
@@ +249,3 @@
+      llvm::errs() << "SymbolTable: error while merging " << atom.name()
+                   << "\n";
+      llvm::report_fatal_error("duplicate symbol error");
----------------
Rui Ueyama wrote:
> This can be at end of the previous line.
formatted using clang-format else goes above 80 cols. 

================
Comment at: lib/Core/Resolver.cpp:255
@@ -235,3 +254,3 @@
     _symbolTable.add(atom);
 
   if (_context.deadStrip()) {
----------------
Rui Ueyama wrote:
> Add '}' to this line and '{' at line 253.
ok.


http://llvm-reviews.chandlerc.com/D3242