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