andcarminati added inline comments.
================ Comment at: lldb/source/Symbol/ObjectFile.cpp:559 + lldb::offset_t section_offset) const { + offset_t offset = section->GetOffset() + section_offset; + return m_data.GetCStr(&offset); ---------------- Wouldn't it be better to use ``` section->GetFileOffset() ``` to access m_data? As this object contains data loaded from the file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120578/new/ https://reviews.llvm.org/D120578 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits