[PATCH 1/2] covoar: Add symbol set reader and ELF data parser to covoar.

Chris Johns chrisj at rtems.org
Fri Apr 27 03:48:21 UTC 2018


On 27/04/2018 08:51, Joel Sherrill wrote:
> 
>        // Create the set of desired symbols.
>        SymbolsToAnalyze = new Coverage::DesiredSymbols();
>     -  SymbolsToAnalyze->load( symbolsFile );
> 
> 
> I am having trouble seeing what happened to the DesiredSymbols class. Is
> it no longer needed? Did we change the class from which SymbolsToAnalyze
> is derived but provide all the same needed interfaces?
> 
> I'm having trouble seeing how the new class fits in. Help me.

I am adding support to determine the build top from the executable path(s) and
to verify the build path, arch prefix and BSP match for all executables passed
in and then I will load the symbols from the libraries listed in the config file
directly into the desired symbols class. It looks me like these extra classes
are not needed.

Chris




More information about the devel mailing list