[PATCH] [lld][core] support .gnu.linkonce sections
Shankar Kalpathi Easwaran
shankarke at gmail.com
Mon Mar 31 20:10:41 PDT 2014
================
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
More information about the llvm-commits
mailing list