| Message ID | 20250212153002.3607542-1-j@lambda.is |
|---|---|
| Headers |
Return-Path: <gcc-patches-bounces~patchwork=sourceware.org@gcc.gnu.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id AA4493858408 for <patchwork@sourceware.org>; Wed, 12 Feb 2025 15:32:10 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org AA4493858408 X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from mx.kolabnow.com (mx.kolabnow.com [212.103.80.154]) by sourceware.org (Postfix) with ESMTPS id EA4A13858410 for <gcc-patches@gcc.gnu.org>; Wed, 12 Feb 2025 15:30:19 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org EA4A13858410 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=lambda.is Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=lambda.is ARC-Filter: OpenARC Filter v1.0.0 sourceware.org EA4A13858410 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=212.103.80.154 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1739374221; cv=none; b=kFg8hepsxnymlXeHkfr8S8tXBGunKZJOM/9pDcHCKwKS10HLnlicXKht7F9GlLunjtzTX7rtCSqRgKJIGLSilcICdlyI38Xzy9F2nsoZvxhg9kPLQ754QiUrOBeFVpOGlrm+D6eu6o8v4y5N45si3Ovz44nAJUGrb1RVtiV2IJY= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1739374221; c=relaxed/simple; bh=WlFKzUJkKp9MEQsejfkuJT8urh8LFvnhi+xWVsTVDnw=; h=DKIM-Signature:From:To:Subject:Date:Message-Id:MIME-Version; b=k3n8qNiP4+H/qEGVhSUSaVm+BLnnMyL9z6WgcRBJi09zYpdERmp2ZbdTz/rLicozqUodDT3aXyPIh2tz7DYGoyq3oiddwXgm7U/lvdxS0MjZ6NelaCZWWAAigiGP2TryuJ6A5VVKd0EKnVNivB0mzX6+NggtFa4WYBx21/1gKYw= ARC-Authentication-Results: i=1; server2.sourceware.org DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org EA4A13858410 Received: from localhost (unknown [127.0.0.1]) by mx.kolabnow.com (Postfix) with ESMTP id D102E3005006; Wed, 12 Feb 2025 16:30:18 +0100 (CET) Authentication-Results: ext-mx-out013.mykolab.com (amavis); dkim=pass reason="pass (just generated, assumed good)" header.d=lambda.is DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lambda.is; h= content-transfer-encoding:content-type:content-type:mime-version :message-id:date:date:subject:subject:from:from:received :received:received; s=dkim2; t=1739374216; x=1741188617; bh=Q4Q3 VDDMNp/KFZpw1kC/O1JpeWu9mVrQDeK3v0iV8Vs=; b=P68gp0wnBzCgJXAt4WZk 0Q7Rfvict6NQWh6b/mdEozaAQDP4Yv8kAWMRS6kMrrxuyc68X2akHyhj6Yed73AO 4LFsXGi8u1H4hRPWdh782HyiusvVL8okxiko7hBGxPvgaCKlWjYwn4WvFH+knGKa cap/Di/Ch2JSh3W4pCr3jNsmstHm9+qX8YN1BR9JwfRH/IrAUO9MTEfrJa1kqi0d Yhvpj8nim+2Ilazyz38/Zaw42DSpkVEoGgy79jTeh8NNzFsOhJEUjvFhEnbYmONB 8zCdX0FRWsCo74NMSV399SWx0Uu2cojMMh5RwPEEuAai/tfQ4wGWHZYWsLMWWf+h Qw== X-Virus-Scanned: amavis at mykolab.com X-Spam-Score: -0.999 X-Spam-Level: X-Spam-Status: No, score=3.9 required=5.0 tests=BAYES_00, DKIM_INVALID, DKIM_SIGNED, KAM_DMARC_STATUS, RCVD_IN_BL_SPAMCOP_NET, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=no autolearn_force=no version=3.4.6 Received: from mx.kolabnow.com ([127.0.0.1]) by localhost (ext-mx-out013.mykolab.com [127.0.0.1]) (amavis, port 10024) with ESMTP id gzVk9Q74eFQG; Wed, 12 Feb 2025 16:30:16 +0100 (CET) Received: from int-mx011.mykolab.com (unknown [10.9.13.11]) by mx.kolabnow.com (Postfix) with ESMTPS id 178663005000; Wed, 12 Feb 2025 16:30:14 +0100 (CET) Received: from ext-subm010.mykolab.com (unknown [10.9.6.10]) by int-mx011.mykolab.com (Postfix) with ESMTPS id BA37C317CC68; Wed, 12 Feb 2025 16:30:14 +0100 (CET) From: =?utf-8?q?J=C3=B8rgen_Kvalsvik?= <j@lambda.is> To: gcc-patches@gcc.gnu.org Cc: hubicka@ucw.cz, =?utf-8?q?J=C3=B8rgen_Kvalsvik?= <j@lambda.is> Subject: [PATCH 0/2] v2 Add prime path coverage to gcc/gcov Date: Wed, 12 Feb 2025 16:30:00 +0100 Message-Id: <20250212153002.3607542-1-j@lambda.is> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Spam-Level: *** X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Gcc-patches mailing list <gcc-patches.gcc.gnu.org> List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe> List-Archive: <https://gcc.gnu.org/pipermail/gcc-patches/> List-Post: <mailto:gcc-patches@gcc.gnu.org> List-Help: <mailto:gcc-patches-request@gcc.gnu.org?subject=help> List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe> Errors-To: gcc-patches-bounces~patchwork=sourceware.org@gcc.gnu.org |
| Series |
v2 Add prime path coverage to gcc/gcov
|
|
Message
Jørgen Kvalsvik
Feb. 12, 2025, 3:30 p.m. UTC
I have applied fixes for everything in the last review, plus some GNU style fixes that I had missed previously. We have tested and used a build with this applied for 3-4 months now and haven't run into any issues. Jørgen Kvalsvik (2): gcov: branch, conds, calls in function summaries Add prime path coverage to gcc/gcov gcc/Makefile.in | 6 +- gcc/builtins.cc | 2 +- gcc/collect2.cc | 6 +- gcc/common.opt | 16 + gcc/doc/gcov.texi | 187 +++ gcc/doc/invoke.texi | 36 + gcc/gcc.cc | 4 +- gcc/gcov-counter.def | 3 + gcc/gcov-io.h | 3 + gcc/gcov.cc | 535 +++++- gcc/ipa-inline.cc | 2 +- gcc/passes.cc | 4 +- gcc/path-coverage.cc | 776 +++++++++ gcc/prime-paths.cc | 2052 ++++++++++++++++++++++++ gcc/profile.cc | 6 +- gcc/selftest-run-tests.cc | 1 + gcc/selftest.h | 1 + gcc/testsuite/g++.dg/gcov/gcov-22.C | 170 ++ gcc/testsuite/g++.dg/gcov/gcov-23-1.h | 9 + gcc/testsuite/g++.dg/gcov/gcov-23-2.h | 9 + gcc/testsuite/g++.dg/gcov/gcov-23.C | 30 + gcc/testsuite/gcc.misc-tests/gcov-29.c | 869 ++++++++++ gcc/testsuite/gcc.misc-tests/gcov-30.c | 869 ++++++++++ gcc/testsuite/gcc.misc-tests/gcov-31.c | 35 + gcc/testsuite/gcc.misc-tests/gcov-32.c | 24 + gcc/testsuite/gcc.misc-tests/gcov-33.c | 27 + gcc/testsuite/gcc.misc-tests/gcov-34.c | 29 + gcc/testsuite/lib/gcov.exp | 118 +- gcc/tree-profile.cc | 11 +- 29 files changed, 5818 insertions(+), 22 deletions(-) create mode 100644 gcc/path-coverage.cc create mode 100644 gcc/prime-paths.cc create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-22.C create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-1.h create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-2.h create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23.C create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-29.c create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-30.c create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-31.c create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-32.c create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-33.c create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-34.c
Comments
Ping On 2/12/25 16:30, Jørgen Kvalsvik wrote: > I have applied fixes for everything in the last review, plus some GNU > style fixes that I had missed previously. We have tested and used a > build with this applied for 3-4 months now and haven't run into any > issues. > > Jørgen Kvalsvik (2): > gcov: branch, conds, calls in function summaries > Add prime path coverage to gcc/gcov > > gcc/Makefile.in | 6 +- > gcc/builtins.cc | 2 +- > gcc/collect2.cc | 6 +- > gcc/common.opt | 16 + > gcc/doc/gcov.texi | 187 +++ > gcc/doc/invoke.texi | 36 + > gcc/gcc.cc | 4 +- > gcc/gcov-counter.def | 3 + > gcc/gcov-io.h | 3 + > gcc/gcov.cc | 535 +++++- > gcc/ipa-inline.cc | 2 +- > gcc/passes.cc | 4 +- > gcc/path-coverage.cc | 776 +++++++++ > gcc/prime-paths.cc | 2052 ++++++++++++++++++++++++ > gcc/profile.cc | 6 +- > gcc/selftest-run-tests.cc | 1 + > gcc/selftest.h | 1 + > gcc/testsuite/g++.dg/gcov/gcov-22.C | 170 ++ > gcc/testsuite/g++.dg/gcov/gcov-23-1.h | 9 + > gcc/testsuite/g++.dg/gcov/gcov-23-2.h | 9 + > gcc/testsuite/g++.dg/gcov/gcov-23.C | 30 + > gcc/testsuite/gcc.misc-tests/gcov-29.c | 869 ++++++++++ > gcc/testsuite/gcc.misc-tests/gcov-30.c | 869 ++++++++++ > gcc/testsuite/gcc.misc-tests/gcov-31.c | 35 + > gcc/testsuite/gcc.misc-tests/gcov-32.c | 24 + > gcc/testsuite/gcc.misc-tests/gcov-33.c | 27 + > gcc/testsuite/gcc.misc-tests/gcov-34.c | 29 + > gcc/testsuite/lib/gcov.exp | 118 +- > gcc/tree-profile.cc | 11 +- > 29 files changed, 5818 insertions(+), 22 deletions(-) > create mode 100644 gcc/path-coverage.cc > create mode 100644 gcc/prime-paths.cc > create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-22.C > create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-1.h > create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-2.h > create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23.C > create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-29.c > create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-30.c > create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-31.c > create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-32.c > create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-33.c > create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-34.c >
Ping. On 2/21/25 19:15, Jørgen Kvalsvik wrote: > Ping > > On 2/12/25 16:30, Jørgen Kvalsvik wrote: >> I have applied fixes for everything in the last review, plus some GNU >> style fixes that I had missed previously. We have tested and used a >> build with this applied for 3-4 months now and haven't run into any >> issues. >> >> Jørgen Kvalsvik (2): >> gcov: branch, conds, calls in function summaries >> Add prime path coverage to gcc/gcov >> >> gcc/Makefile.in | 6 +- >> gcc/builtins.cc | 2 +- >> gcc/collect2.cc | 6 +- >> gcc/common.opt | 16 + >> gcc/doc/gcov.texi | 187 +++ >> gcc/doc/invoke.texi | 36 + >> gcc/gcc.cc | 4 +- >> gcc/gcov-counter.def | 3 + >> gcc/gcov-io.h | 3 + >> gcc/gcov.cc | 535 +++++- >> gcc/ipa-inline.cc | 2 +- >> gcc/passes.cc | 4 +- >> gcc/path-coverage.cc | 776 +++++++++ >> gcc/prime-paths.cc | 2052 ++++++++++++++++++++++++ >> gcc/profile.cc | 6 +- >> gcc/selftest-run-tests.cc | 1 + >> gcc/selftest.h | 1 + >> gcc/testsuite/g++.dg/gcov/gcov-22.C | 170 ++ >> gcc/testsuite/g++.dg/gcov/gcov-23-1.h | 9 + >> gcc/testsuite/g++.dg/gcov/gcov-23-2.h | 9 + >> gcc/testsuite/g++.dg/gcov/gcov-23.C | 30 + >> gcc/testsuite/gcc.misc-tests/gcov-29.c | 869 ++++++++++ >> gcc/testsuite/gcc.misc-tests/gcov-30.c | 869 ++++++++++ >> gcc/testsuite/gcc.misc-tests/gcov-31.c | 35 + >> gcc/testsuite/gcc.misc-tests/gcov-32.c | 24 + >> gcc/testsuite/gcc.misc-tests/gcov-33.c | 27 + >> gcc/testsuite/gcc.misc-tests/gcov-34.c | 29 + >> gcc/testsuite/lib/gcov.exp | 118 +- >> gcc/tree-profile.cc | 11 +- >> 29 files changed, 5818 insertions(+), 22 deletions(-) >> create mode 100644 gcc/path-coverage.cc >> create mode 100644 gcc/prime-paths.cc >> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-22.C >> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-1.h >> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-2.h >> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23.C >> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-29.c >> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-30.c >> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-31.c >> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-32.c >> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-33.c >> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-34.c >> >
Ping. On 3/5/25 20:21, Jørgen Kvalsvik wrote: > Ping. > > On 2/21/25 19:15, Jørgen Kvalsvik wrote: >> Ping >> >> On 2/12/25 16:30, Jørgen Kvalsvik wrote: >>> I have applied fixes for everything in the last review, plus some GNU >>> style fixes that I had missed previously. We have tested and used a >>> build with this applied for 3-4 months now and haven't run into any >>> issues. >>> >>> Jørgen Kvalsvik (2): >>> gcov: branch, conds, calls in function summaries >>> Add prime path coverage to gcc/gcov >>> >>> gcc/Makefile.in | 6 +- >>> gcc/builtins.cc | 2 +- >>> gcc/collect2.cc | 6 +- >>> gcc/common.opt | 16 + >>> gcc/doc/gcov.texi | 187 +++ >>> gcc/doc/invoke.texi | 36 + >>> gcc/gcc.cc | 4 +- >>> gcc/gcov-counter.def | 3 + >>> gcc/gcov-io.h | 3 + >>> gcc/gcov.cc | 535 +++++- >>> gcc/ipa-inline.cc | 2 +- >>> gcc/passes.cc | 4 +- >>> gcc/path-coverage.cc | 776 +++++++++ >>> gcc/prime-paths.cc | 2052 ++++++++++++++++++++++++ >>> gcc/profile.cc | 6 +- >>> gcc/selftest-run-tests.cc | 1 + >>> gcc/selftest.h | 1 + >>> gcc/testsuite/g++.dg/gcov/gcov-22.C | 170 ++ >>> gcc/testsuite/g++.dg/gcov/gcov-23-1.h | 9 + >>> gcc/testsuite/g++.dg/gcov/gcov-23-2.h | 9 + >>> gcc/testsuite/g++.dg/gcov/gcov-23.C | 30 + >>> gcc/testsuite/gcc.misc-tests/gcov-29.c | 869 ++++++++++ >>> gcc/testsuite/gcc.misc-tests/gcov-30.c | 869 ++++++++++ >>> gcc/testsuite/gcc.misc-tests/gcov-31.c | 35 + >>> gcc/testsuite/gcc.misc-tests/gcov-32.c | 24 + >>> gcc/testsuite/gcc.misc-tests/gcov-33.c | 27 + >>> gcc/testsuite/gcc.misc-tests/gcov-34.c | 29 + >>> gcc/testsuite/lib/gcov.exp | 118 +- >>> gcc/tree-profile.cc | 11 +- >>> 29 files changed, 5818 insertions(+), 22 deletions(-) >>> create mode 100644 gcc/path-coverage.cc >>> create mode 100644 gcc/prime-paths.cc >>> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-22.C >>> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-1.h >>> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23-2.h >>> create mode 100644 gcc/testsuite/g++.dg/gcov/gcov-23.C >>> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-29.c >>> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-30.c >>> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-31.c >>> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-32.c >>> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-33.c >>> create mode 100644 gcc/testsuite/gcc.misc-tests/gcov-34.c >>> >> >