RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 22:27:35 +00:00
Code Issues Activity Actions

LibDebug: Store optional parent_offset in Dwarf::DIE objects

Browse source 
In the current implementation, only DIE objects that are created via
DIE::for_each_child() will have parent offsets.

DIE objects that are created with CompilationUnit::get_die_at_offset()
do not currently store a parent offset.

We may improve this in the future, but this is enough for what we
currently need.
This commit is contained in:
Itamar 2021-06-19 11:39:51 +03:00 • committed by Andreas Kling
parent 84609aecc1
commit a5f69efa5c
2 changed files with 9 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

11
Userland/Libraries/LibDebug/Dwarf/DIE.cpp
View file

@ -1,5 +1,5 @@
/*
* Copyright (c) 2020, Itamar S. <itamar8910@gmail.com>
* Copyright (c) 2020-2021, Itamar S. <itamar8910@gmail.com>
*
* SPDX-License-Identifier: BSD-2-Clause
*/
@ -12,7 +12,7 @@
namespace Debug::Dwarf {
DIE::DIE(const CompilationUnit& unit, u32 offset)
DIE::DIE(const CompilationUnit& unit, u32 offset, Optional<u32> parent_offset)
: m_compilation_unit(unit)
, m_offset(offset)
{
@ -37,6 +37,7 @@ DIE::DIE(const CompilationUnit& unit, u32 offset)
}
}
m_size = stream.offset() - m_offset;
m_parent_offset = parent_offset;
}
Optional<AttributeValue> DIE::get_attribute(const Attribute& attribute) const
@ -61,13 +62,13 @@ void DIE::for_each_child(Function<void(const DIE& child)> callback) const
if (!m_has_children)
return;
NonnullOwnPtr<DIE> current_child = make<DIE>(m_compilation_unit, m_offset + m_size);
NonnullOwnPtr<DIE> current_child = make<DIE>(m_compilation_unit, m_offset + m_size, m_offset);
while (true) {
callback(*current_child);
if (current_child->is_null())
break;
if (!current_child->has_children()) {
current_child = make<DIE>(m_compilation_unit, current_child->offset() + current_child->size());
current_child = make<DIE>(m_compilation_unit, current_child->offset() + current_child->size(), m_offset);
continue;
}
@ -84,7 +85,7 @@ void DIE::for_each_child(Function<void(const DIE& child)> callback) const
sibling_offset = sub_child.offset() + sub_child.size();
});
}
current_child = make<DIE>(m_compilation_unit, sibling_offset);
current_child = make<DIE>(m_compilation_unit, sibling_offset, m_offset);
}
}