From: Ian Lance Taylor <iant@google.com>
Date: Wed, 10 Oct 2007 06:33:56 +0000 (+0000)
Subject: From Craig Silverstein: don't get confused if the same file name
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=cec9d2f362a9f1c13aad15caed66493463efb674;p=binutils-gdb.git

From Craig Silverstein: don't get confused if the same file name
occurs in an archive.
---

diff --git a/gold/merge.cc b/gold/merge.cc
index 0db62ef7c82..75cbc18666b 100644
--- a/gold/merge.cc
+++ b/gold/merge.cc
@@ -40,7 +40,15 @@ Output_merge_base::Merge_key_less::operator()(const Merge_key& mk1,
   // matter.  We want to get consistent results across links so we
   // don't use pointer comparison.
   if (mk1.object != mk2.object)
-    return mk1.object->name() < mk2.object->name();
+    {
+      // Two different object files can have the same name: if foo.a
+      // includes both bar/qux.o and baz/qux.o, then both end up with
+      // the name foo.a(qux.o).  But it's impossible for two different
+      // object files to have both the same name and the same offset.
+      if (mk1.object->offset() != mk2.object->offset())
+        return mk1.object->offset() < mk2.object->offset();
+      return mk1.object->name() < mk2.object->name();
+    }
   if (mk1.shndx != mk2.shndx)
     return mk1.shndx < mk2.shndx;
   return mk1.offset < mk2.offset;
diff --git a/gold/object.h b/gold/object.h
index 2baf2da87a6..8f0a3b7726e 100644
--- a/gold/object.h
+++ b/gold/object.h
@@ -130,6 +130,11 @@ class Object
   name() const
   { return this->name_; }
 
+  // Get the offset into the file.
+  off_t
+  offset() const
+  { return this->offset_; }
+
   // Return whether this is a dynamic object.
   bool
   is_dynamic() const
@@ -277,11 +282,6 @@ class Object
   input_file() const
   { return this->input_file_; }
 
-  // Get the offset into the file.
-  off_t
-  offset() const
-  { return this->offset_; }
-
   // Get a view into the underlying file.
   const unsigned char*
   get_view(off_t start, off_t size, bool cache)