Skip to content

Commit 4d6c333

Browse files
Merge pull request swiftlang#7917 from rastogishubham/NFCChangeStable
Cherry-pick PR swiftlang#7911 into stable/20230725
2 parents 1eeb3fa + f495eff commit 4d6c333

File tree

1 file changed

+13
-25
lines changed

1 file changed

+13
-25
lines changed

llvm/lib/MCCAS/MCCASObjectV1.cpp

Lines changed: 13 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -3107,27 +3107,6 @@ struct DIEVisitor {
31073107
SmallVector<AbbrevContent> AbbrevContents;
31083108
};
31093109

3110-
DIEVisitor() = delete;
3111-
3112-
DIEVisitor(ArrayRef<StringRef> AbbrevEntries,
3113-
BinaryStreamReader DistinctReader, StringRef DistinctData,
3114-
std::function<void(StringRef)> HeaderCallback,
3115-
std::function<void(dwarf::Tag, uint64_t)> StartTagCallback,
3116-
std::function<void(dwarf::Attribute, dwarf::Form, StringRef, bool)>
3117-
AttrCallback,
3118-
std::function<void(bool)> EndTagCallback,
3119-
std::function<void(StringRef)> NewBlockCallback)
3120-
: AbbrevEntries(AbbrevEntries), DistinctReader(DistinctReader),
3121-
DistinctData(DistinctData), HeaderCallback(HeaderCallback),
3122-
StartTagCallback(StartTagCallback), AttrCallback(AttrCallback),
3123-
EndTagCallback(EndTagCallback), NewBlockCallback(NewBlockCallback) {
3124-
AbbrevEntryCache.reserve(AbbrevEntries.size());
3125-
for (unsigned I = 0; I < AbbrevEntries.size(); I++) {
3126-
if (Error E = materializeAbbrevDIE(encodeAbbrevIndex(I)))
3127-
report_fatal_error(std::move(E));
3128-
}
3129-
}
3130-
31313110
Error visitDIERef(DIEDedupeTopLevelRef Ref);
31323111
Error visitDIERef(ArrayRef<DIEDataRef> &DIEChildrenStack);
31333112
Error visitDIEAttrs(BinaryStreamReader &DataReader, StringRef DIEData,
@@ -3312,6 +3291,10 @@ static void popStack(BinaryStreamReader &Reader, StringRef &Data,
33123291
// Visit DIERef CAS objects and materialize them.
33133292
Error DIEVisitor::visitDIERef(ArrayRef<DIEDataRef> &DIEChildrenStack) {
33143293

3294+
for (unsigned I = 0; I < AbbrevEntries.size(); I++)
3295+
if (Error E = materializeAbbrevDIE(encodeAbbrevIndex(I)))
3296+
return E;
3297+
33153298
std::stack<std::pair<StringRef, unsigned>> StackOfNodes;
33163299
auto Data = DIEChildrenStack.empty() ? StringRef()
33173300
: DIEChildrenStack.front().getData();
@@ -3439,9 +3422,14 @@ Error mccasformats::v1::visitDebugInfo(
34393422
HeaderCallback(toStringRef(HeaderData));
34403423

34413424
append_range(TotAbbrevEntries, LoadedTopRef->AbbrevEntries);
3442-
DIEVisitor Visitor{TotAbbrevEntries, DistinctReader,
3443-
UncompressedDistinctData, HeaderCallback,
3444-
StartTagCallback, AttrCallback,
3445-
EndTagCallback, NewBlockCallback};
3425+
DIEVisitor Visitor{{},
3426+
TotAbbrevEntries,
3427+
DistinctReader,
3428+
UncompressedDistinctData,
3429+
HeaderCallback,
3430+
StartTagCallback,
3431+
AttrCallback,
3432+
EndTagCallback,
3433+
NewBlockCallback};
34463434
return Visitor.visitDIERef(LoadedTopRef->RootDIE);
34473435
}

0 commit comments

Comments
 (0)