<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 14, 2021, 7:27 AM Sebastian Huber <<a href="mailto:sebastian.huber@embedded-brains.de">sebastian.huber@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 14/07/2021 14:25, Joel Sherrill wrote:<br>
> <br>
> <br>
> On Wed, Jul 14, 2021, 7:15 AM Sebastian Huber <br>
> <<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a> <br>
> <mailto:<a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a>>> wrote:<br>
> <br>
>     Hello,<br>
> <br>
>     I had a bit of time to work on the gcov support for RTEMS:<br>
> <br>
>     <a href="https://gcc.gnu.org/pipermail/gcc-patches/2021-July/575122.html" rel="noreferrer noreferrer" target="_blank">https://gcc.gnu.org/pipermail/gcc-patches/2021-July/575122.html</a><br>
>     <<a href="https://gcc.gnu.org/pipermail/gcc-patches/2021-July/575122.html" rel="noreferrer noreferrer" target="_blank">https://gcc.gnu.org/pipermail/gcc-patches/2021-July/575122.html</a>><br>
> <br>
>     I tried to add an RTEMS_COVERAGE build option to enable gcov. Then I<br>
>     noticed that such a define is already used in RTEMS:<br>
> <br>
>     testsuites/sptests/spcoverage/init.c:  #if defined(RTEMS_COVERAGE)<br>
> <br>
>     Who provides this define?<br>
> <br>
> <br>
> I have no idea what this does. The covoar reports require no special <br>
> defines. The most you might do is use -Os to make the analysis easier <br>
> since the code is easier to follow when optimising for size.<br>
> <br>
> What does that test even do?<br>
<br>
rtems_task Init(<br>
   rtems_task_argument ignored<br>
)<br>
{<br>
   TEST_BEGIN();<br>
<br>
   puts( "Init - If coverage enabled, call coverage marker methods" );<br>
   #if defined(RTEMS_COVERAGE)<br>
     start_coverage();<br>
     end_coverage();<br>
   #endif<br>
<br>
   TEST_END();<br>
   rtems_test_exit(0);<br>
}<br>
<br>
 From looking at the git log, the start_coverage() and end_coverage() <br>
were never defined in RTEMS. I guess we can remove this test program.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I've honestly got no idea. Removing it seems right.</div><div dir="auto"><br></div><div dir="auto">--joel</div><div dir="auto"><br></div><div dir="auto"><br></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>
embedded brains GmbH<br>
Herr Sebastian HUBER<br>
Dornierstr. 4<br>
82178 Puchheim<br>
Germany<br>
email: <a href="mailto:sebastian.huber@embedded-brains.de" target="_blank" rel="noreferrer">sebastian.huber@embedded-brains.de</a><br>
phone: +49-89-18 94 741 - 16<br>
fax:   +49-89-18 94 741 - 08<br>
<br>
Registergericht: Amtsgericht München<br>
Registernummer: HRB 157899<br>
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler<br>
Unsere Datenschutzerklärung finden Sie hier:<br>
<a href="https://embedded-brains.de/datenschutzerklaerung/" rel="noreferrer noreferrer" target="_blank">https://embedded-brains.de/datenschutzerklaerung/</a><br>
</blockquote></div></div></div>