Mips target in gold - part 3
Cary Coutant
ccoutant@google.com
Sat Dec 21 00:12:00 GMT 2013
Sorry it's taken me so long to review this patch. I'll start with the
smaller patches...
+2013-08-02 Sasa Stankovic <Sasa.Stankovic@imgtec.com>
+
+ * symtab.cc (Symbol_table::set_dynsym_indexes): Allow a target to set
+ dynsym indexes.
+ * target.h (Target::has_custom_set_dynsym_indexes): New function.
+ (Target::set_dynsym_indexes): New function.
--- a/gold/symtab.cc
+++ b/gold/symtab.cc
@@ -2369,6 +2369,25 @@ Symbol_table::set_dynsym_indexes(unsigned int index,
Versions* versions)
{
std::vector<Symbol*> as_needed_sym;
+ std::vector<Symbol*> dyn_symbols;
dyn_symbols should be local to the if-block below.
--- a/gold/target.h
+++ b/gold/target.h
@@ -37,6 +37,7 @@
#include "elfcpp.h"
#include "options.h"
#include "parameters.h"
+#include "stringpool.h"
You shouldn't need to include "stringpool.h". The set_dynsym_indexes
function takes a Stringpool*, so you only need a forward declaration
for class Stringpool.
This patch is OK with those changes.
-cary
More information about the Binutils
mailing list