Rename MergeInputSection::getOffset.
authorRafael Espindola <rafael.espindola@gmail.com>
Thu, 19 Apr 2018 16:05:07 +0000 (16:05 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Thu, 19 Apr 2018 16:05:07 +0000 (16:05 +0000)
Unlike the getOffset in the base class, this one computes the offset
in the parent synthetic section, not the final output section.

llvm-svn: 330339

lld/ELF/InputSection.cpp
lld/ELF/InputSection.h

index fbbd370..f094be5 100644 (file)
@@ -156,8 +156,9 @@ uint64_t SectionBase::getOffset(uint64_t Offset) const {
   case Merge:
     const MergeInputSection *MS = cast<MergeInputSection>(this);
     if (InputSection *IS = MS->getParent())
-      return cast<InputSection>(IS->Repl)->OutSecOff + MS->getOffset(Offset);
-    return MS->getOffset(Offset);
+      return cast<InputSection>(IS->Repl)->OutSecOff +
+             MS->getParentOffset(Offset);
+    return MS->getParentOffset(Offset);
   }
   llvm_unreachable("invalid section kind");
 }
@@ -972,7 +973,7 @@ SectionPiece *MergeInputSection::getSectionPiece(uint64_t Offset) {
 // Returns the offset in an output section for a given input offset.
 // Because contents of a mergeable section is not contiguous in output,
 // it is not just an addition to a base output offset.
-uint64_t MergeInputSection::getOffset(uint64_t Offset) const {
+uint64_t MergeInputSection::getParentOffset(uint64_t Offset) const {
   // Find a string starting at a given offset.
   auto It = OffsetMap.find(Offset);
   if (It != OffsetMap.end())
index 1f0b37b..db4d672 100644 (file)
@@ -229,9 +229,9 @@ public:
       LiveOffsets.insert(Offset);
   }
 
-  // Translate an offset in the input section to an offset
-  // in the output section.
-  uint64_t getOffset(uint64_t Offset) const;
+  // Translate an offset in the input section to an offset in the parent
+  // MergeSyntheticSection.
+  uint64_t getParentOffset(uint64_t Offset) const;
 
   // Splittable sections are handled as a sequence of data
   // rather than a single large blob of data.