Sure, thanks for replying quickly, let me update readme.md file in my next patch.
-----Original Message----- From: Kinney, Michael D <[email protected]> Sent: Tuesday, September 13, 2022 11:10 PM To: Guo, Gua <[email protected]>; [email protected]; Kinney, Michael D <[email protected]> Cc: Sean Brogan <[email protected]>; Barkelew, Bret <[email protected]>; Michael Kubacki <[email protected]> Subject: RE: [PATCH] UnitTestFrameworkPkg: Add code coverage support for GCC Hi, This change looks safe because it is scoped to host-based unit tests that include this file in the DSC file that builds host-based tests. Can you also update the UnitTestFrameworkPkg/ReadMe.md to describe how a developer can enable/use the code coverage feature for GCC host-based tests? Mike > -----Original Message----- > From: Guo, Gua <[email protected]> > Sent: Tuesday, September 13, 2022 6:24 AM > To: [email protected] > Cc: Guo, Gua <[email protected]>; Kinney, Michael D > <[email protected]>; Sean Brogan <[email protected]>; > Barkelew, Bret <[email protected]>; Michael Kubacki > <[email protected]> > Subject: [PATCH] UnitTestFrameworkPkg: Add code coverage support for > GCC > > From: Gua Guo <[email protected]> > > In order to collect code coverage after running executable file, > generate *.gcda and *.gcno file that require by lcov tool to generate > code coverage report. > > Cc: Michael D Kinney <[email protected]> > Cc: Sean Brogan <[email protected]> > Cc: Bret Barkelew <[email protected]> > Cc: Michael Kubacki <[email protected]> > Signed-off-by: Gua Guo <[email protected]> > --- > UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc > b/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc > index 4dd8d4ac67..8623aecb33 100644 > --- a/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc > +++ b/UnitTestFrameworkPkg/UnitTestFrameworkPkgHost.dsc.inc > @@ -19,7 +19,7 @@ > > MemoryAllocationLib|UnitTestFrameworkPkg/Library/Posix/MemoryAllocatio > nLibPosix/MemoryAllocationLibPosix.inf > > > > [BuildOptions] > > - GCC:*_*_*_CC_FLAGS = -fno-pie > > + GCC:*_*_*_CC_FLAGS = -fno-pie -ftest-coverage -fprofile-arcs > > !ifdef $(UNIT_TESTING_DEBUG) > > MSFT:*_*_*_CC_FLAGS = -D UNIT_TESTING_DEBUG=1 > > GCC:*_*_*_CC_FLAGS = -D UNIT_TESTING_DEBUG=1 > > -- > 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#93719): https://edk2.groups.io/g/devel/message/93719 Mute This Topic: https://groups.io/mt/93654981/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
