Fix bug with exponentially long build times

apple · Oct 11, 2022 · 5b62745 · 5b62745
1 parent 8c46740
commit 5b62745
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/llvm/include/llvm/MC/CAS/MCCASObjectV1.h b/llvm/include/llvm/MC/CAS/MCCASObjectV1.h
@@ -649,7 +649,7 @@ class DebugInfoSectionRef : public SpecificRef<DebugInfoSectionRef> {
   }
   Expected<uint64_t>
   materialize(MCCASReader &Reader, ArrayRef<char> AbbrevSectionContents,
-              DenseMap<cas::ObjectRef, unsigned> MapOfStringOffsets,
+              DenseMap<cas::ObjectRef, unsigned> &MapOfStringOffsets,
               raw_ostream *Stream = nullptr) const;
 
 private:

diff --git a/llvm/lib/MC/MCCASObjectV1.cpp b/llvm/lib/MC/MCCASObjectV1.cpp
@@ -977,7 +977,7 @@ static Expected<uint64_t> getFormSize(dwarf::Form FormVal, dwarf::FormParams FP,
 static Error
 materializeCUDie(DWARFCompileUnit &DCU, MutableArrayRef<char> SectionContents,
                  StringRef CUData, ArrayRef<char> DistinctDataArrayRef,
-                 DenseMap<cas::ObjectRef, unsigned> MapOfStringOffsets,
+                 DenseMap<cas::ObjectRef, unsigned> &MapOfStringOffsets,
                  ArrayRef<cas::ObjectRef> DebugStringRefsVec,
                  uint64_t &CUOffset, uint64_t &DistinctDataOffset,
                  uint64_t &SectionOffset, unsigned &StrpCount,
@@ -1064,7 +1064,7 @@ materializeCUDie(DWARFCompileUnit &DCU, MutableArrayRef<char> SectionContents,
 
 Expected<uint64_t> DebugInfoSectionRef::materialize(
     MCCASReader &Reader, ArrayRef<char> AbbrevSectionContents,
-    DenseMap<cas::ObjectRef, unsigned> MapOfStringOffsets,
+    DenseMap<cas::ObjectRef, unsigned> &MapOfStringOffsets,
     raw_ostream *Stream) const {
   // Start a new section for relocations.
   Reader.Relocations.emplace_back();