[llvm] r231530 - [dsymutil] Set linked unit start offset early. NFC.

[Frederic Riss]

Author: friss
Date: Fri Mar  6 17:22:50 2015
New Revision: 231530

URL: http://llvm.org/viewvc/llvm-project?rev=231530&view=rev
Log:
[dsymutil] Set linked unit start offset early. NFC.

The start offset of a linked unit is known before starting to clone
its DIEs. Handling DW_FORM_ref_addr attributes requires that this
offset is set while cloning the unit. Split CompileUnit::computeOffsets()
into setStartOffset() and computeNextUnitOffset() and call them
repsectively before cloning the DIEs and right after.

Modified:
    llvm/trunk/tools/dsymutil/DwarfLinker.cpp

Modified: llvm/trunk/tools/dsymutil/DwarfLinker.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/dsymutil/DwarfLinker.cpp?rev=231530&r1=231529&r2=231530&view=diff
==============================================================================
--- llvm/trunk/tools/dsymutil/DwarfLinker.cpp (original)
+++ llvm/trunk/tools/dsymutil/DwarfLinker.cpp Fri Mar  6 17:22:50 2015
@@ -81,12 +81,13 @@ public:
   uint64_t getStartOffset() const { return StartOffset; }
   uint64_t getNextUnitOffset() const { return NextUnitOffset; }
 
-  /// \brief Set the start and end offsets for this unit. Must be
-  /// called after the CU's DIEs have been cloned. The unit start
-  /// offset will be set to \p DebugInfoSize.
+  void setStartOffset(uint64_t DebugInfoSize) { StartOffset = DebugInfoSize; }
+
+  /// \brief Compute the end offset for this unit. Must be
+  /// called after the CU's DIEs have been cloned.
   /// \returns the next unit offset (which is also the current
   /// debug_info section size).
-  uint64_t computeOffsets(uint64_t DebugInfoSize);
+  uint64_t computeNextUnitOffset();
 
 private:
   DWARFUnit &OrigUnit;
@@ -97,8 +98,7 @@ private:
   uint64_t NextUnitOffset;
 };
 
-uint64_t CompileUnit::computeOffsets(uint64_t DebugInfoSize) {
-  StartOffset = DebugInfoSize;
+uint64_t CompileUnit::computeNextUnitOffset() {
   NextUnitOffset = StartOffset + 11 /* Header size */;
   // The root DIE might be null, meaning that the Unit had nothing to
   // contribute to the linked output. In that case, we will emit the
@@ -1307,10 +1307,11 @@ bool DwarfLinker::link(const DebugMap &M
     if (!ValidRelocs.empty())
       for (auto &CurrentUnit : Units) {
         const auto *InputDIE = CurrentUnit.getOrigUnit().getCompileUnitDIE();
+        CurrentUnit.setStartOffset(OutputDebugInfoSize);
         DIE *OutputDIE =
             cloneDIE(*InputDIE, CurrentUnit, 11 /* Unit Header size */);
         CurrentUnit.setOutputUnitDIE(OutputDIE);
-        OutputDebugInfoSize = CurrentUnit.computeOffsets(OutputDebugInfoSize);
+        OutputDebugInfoSize = CurrentUnit.computeNextUnitOffset();
       }
 
     // Emit all the compile unit's debug information.