<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr">On Thu, May 24, 2018, 7:59 AM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, May 23, 2018 at 11:11 PM, Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</a>> wrote:<br>
><br>
><br>
> On Wed, May 23, 2018, 9:59 PM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank" rel="noreferrer">chrisj@rtems.org</a>> wrote:<br>
>><br>
>> On 24/5/18 12:07 pm, Joel Sherrill wrote:<br>
>> > ---<br>
>> >  tester/covoar/ObjdumpProcessor.cc | 22 ++++++++++++++++++++++<br>
>> >  1 file changed, 22 insertions(+)<br>
>> ><br>
>> > diff --git a/tester/covoar/ObjdumpProcessor.cc<br>
>> > b/tester/covoar/ObjdumpProcessor.cc<br>
>> > index d41906c..183722b 100644<br>
>> > --- a/tester/covoar/ObjdumpProcessor.cc<br>
>> > +++ b/tester/covoar/ObjdumpProcessor.cc<br>
>> > @@ -384,6 +384,28 @@ namespace Coverage {<br>
>> >          break;<br>
>> >        }<br>
>> ><br>
>> > +      // This string indicates a block of padding at the end of a<br>
>> > method<br>
>> > +      if (!strncmp(line.c_str(), "\t...", 4)) {<br>
>><br>
>> You have an std::string so why no use C++ rather than resorting to C?<br>
>><br>
>> For example <a href="http://en.cppreference.com/w/cpp/string/basic_string/compare" rel="noreferrer noreferrer" target="_blank">http://en.cppreference.com/w/cpp/string/basic_string/compare</a>.<br>
><br>
><br>
> I was in a hurry and had trouble getting this logic to trip at all. I still<br>
> fix this. It does need to be like strncmp.<br>
><br>
> The +4 is awful though<br>
Classical technical debt. :)<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I hoped Chris could get the info from the dwarf but he has a similar issue. You see the padding between methods both ways.</div><div dir="auto"><br></div><div dir="auto">When objdump printed a series of nope as padding, I could take the address of the first nop. With ... there is no address and it is harder to figure out.</div><div dir="auto"><br></div><div dir="auto">I think dwarf+embedding our own dump library is the long term replacement so this will keep us moving.</div><div dir="auto"><br></div><div dir="auto">Worst case, I could add code to the target specific code to recognize the previous instruction and get the size. Hmm.. should be a return variant. Let me think on that. Can't be that many instructions that make sense as the last one it a method.</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
>><br>
>><br>
>> Chris<br>
>> _______________________________________________<br>
>> devel mailing list<br>
>> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
>> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
><br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>