<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 12, 2021 at 5:47 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">These are design question and not review issues ....<br>
<br>
On 12/3/21 5:33 am, Alex White wrote:<br>
> +  // Create data based on target.<br>
> +  TargetInfo = Target::TargetFactory( buildTarget );<br>
<br>
Any pointers in this object given there is a copy operator at work here?<br></blockquote><div><br></div><div>There shouldn't be.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +  // Create the set of desired symbols.<br>
> +  SymbolsToAnalyze = new Coverage::DesiredSymbols();<br>
<br>
How hard would it be to convert these types of allocations in covoar to<br>
std::shared_ptr<> or std::unique_ptr<> (if that is suitable or possible)?<br></blockquote><div><br></div><div>We probably need to have a data flow discussion as part of evaluating this.</div><div>When (if) this code is parallellized like Alex and I would love to see done, </div><div>the base information gathering will happen upfront and then the analysis</div><div>and report generation can happen in separate threads for each symbol</div><div>set. This reduces the objdump for sure. I thinkTargetinfo is a shared read-only </div><div>class once created and all threads can share it.</div><div><br></div><div>SymbolSet would become a thread local variable since a thread would be</div><div>created per SymbolSet.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Chris<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>