From: Neil Horman <nhorman@openssl.org>
Date: Mon, 16 Jun 2025 21:35:07 +0000 (-0400)
Subject: Add branch coverage to our coveralls run
X-Git-Tag: openssl-3.6.0-alpha1~572
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1187df53287dcb1ba04479a1cd2dd0547dc14354;p=thirdparty%2Fopenssl.git

Add branch coverage to our coveralls run

Coveralls is capable of reporting not only covered lines in a build, but
also the number of times a given branch was taken vs. not taken.  This
will help us identify locations where we might make better use of the
openssl_likely and openssl_unlikely macros to optimize branch prediction
when building openssl

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <ppzgs1@gmail.com>
(Merged from https://github.com/openssl/openssl/pull/27839)
---

diff --git a/.github/workflows/coveralls.yml b/.github/workflows/coveralls.yml
index 38db01c9c12..c3a1069a224 100644
--- a/.github/workflows/coveralls.yml
+++ b/.github/workflows/coveralls.yml
@@ -116,6 +116,7 @@ jobs:
              --exclude "${PWD}/fuzz/*"
              --exclude "/usr/include/*"
              --ignore-errors mismatch
+             --branch-coverage
               -o ./lcov.info
     - name: Coveralls upload
       uses: coverallsapp/github-action@v2.3.2
diff --git a/Configurations/10-main.conf b/Configurations/10-main.conf
index 6465651507e..01526cd3efd 100644
--- a/Configurations/10-main.conf
+++ b/Configurations/10-main.conf
@@ -680,8 +680,7 @@ my %targets = (
         CFLAGS           => picker(default => "-Wall",
                                    debug   => "-O0 -g",
                                    release => "-O3",
-                                   profiled => "-O3 -fprofile-generate -fprofile-arcs -fprofile-values -ftest-coverage",
-                                   pgo => "-O3 -fprofile-use -Wno-missing-profile"),
+                                   pgo => "-O3 -fprofile-use -Wno-coverage-mismatch"),
         CXXFLAGS         => picker(default => "-Wall",
                                    debug   => "-O0 -g",
                                    release => "-O3"),
diff --git a/Configure b/Configure
index ef6ba49f6bb..d105cfe3307 100755
--- a/Configure
+++ b/Configure
@@ -990,10 +990,6 @@ while (@argvcopy)
                 {
                 $config{build_type} = "debug";
                 }
-        elsif (/^-p$/)
-                {
-                $config{build_type} = "profiled";
-                }
         elsif (/^-v$/)          # From older 'config'
                 {
                 $guess_opts{verbose} = 1;
@@ -1021,10 +1017,6 @@ while (@argvcopy)
                 {
                 $config{build_type} = "release";
                 }
-        elsif (/^--profiled$/)
-                {
-                $config{build_type} = "profiled";
-                }
         elsif (/^--pgo$/)
                 {
                 $config{build_type} = "pgo";