[Lldb-commits] [PATCH] ObjectFileELF::GetSectionHeaderInfo sets arch_spec unnecessarily
tfiala at google.com
Fri Jul 11 08:25:52 PDT 2014
I've checked in the first patch here (minor tweak to comment):
Transmitting file data .
Committed revision 212822.
I'm looking at the second one now.
On Fri, Jul 11, 2014 at 8:14 AM, Todd Fiala <tfiala at google.com> wrote:
> Okay - so you're not aware then of any cases where we build ObjectFileELF
> files incrementally and might need to re-parse the headers to incrementally
> pick up anything new? (I'm not aware of it but just raise it as "Do we do
> this anywhere?").
> On Fri, Jul 11, 2014 at 8:01 AM, Ed Maste <emaste at freebsd.org> wrote:
>> On 11 July 2014 10:50, Todd Fiala <tfiala at google.com> wrote:
>> > I was about to give his second patch a look now. When you looked at
>> it, you were looking at his first one, is that correct? Any thoughts on
>> the second one?
>> That's correct - I was specifically talking about the first patch.
>> I do not believe the second patch will break anything for me -- if we
>> have an ELF file with no sections then m_section_headers.empty() will
>> be true, and we'll just have an additional ParseSectionHeaders() call,
>> as we do today. Either way I think the first patch is correct and
>> that we should apply it, regardless of whether we also apply the
>> second patch. If m_section_headers is not empty() then we must have
>> already parsed the section headers so should not fall back to the
> Todd Fiala | Software Engineer | tfiala at google.com | 650-943-3180
Todd Fiala | Software Engineer | tfiala at google.com | 650-943-3180
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits