From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 23547 invoked by alias); 21 Sep 2017 19:00:58 -0000 Mailing-List: contact elfutils-devel-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Post: List-Help: List-Subscribe: Sender: elfutils-devel-owner@sourceware.org Received: (qmail 23534 invoked by uid 89); 21 Sep 2017 19:00:57 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Checked: by ClamAV 0.99.2 on sourceware.org X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.9 required=5.0 tests=BAYES_00,KAM_LAZY_DOMAIN_SECURITY autolearn=no version=3.3.2 spammy=sad, our, hear, team X-Spam-Status: No, score=-0.9 required=5.0 tests=BAYES_00,KAM_LAZY_DOMAIN_SECURITY autolearn=no version=3.3.2 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on sourceware.org X-Spam-Level: X-HELO: gnu.wildebeest.org Received: from wildebeest.demon.nl (HELO gnu.wildebeest.org) (212.238.236.112) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 21 Sep 2017 19:00:56 +0000 Received: from tarox.wildebeest.org (tarox.wildebeest.org [172.31.17.39]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 1144D302BAA0; Thu, 21 Sep 2017 21:00:54 +0200 (CEST) Received: by tarox.wildebeest.org (Postfix, from userid 1000) id F2F9A40350DE; Thu, 21 Sep 2017 21:00:53 +0200 (CEST) Message-ID: <1506020453.16945.67.camel@klomp.org> Subject: Re: Latest version of dwarflint? From: Mark Wielaard To: Djordje Todorovic , "Ananthakrishna Sowda (asowda)" Cc: "elfutils-devel@sourceware.org" Date: Thu, 21 Sep 2017 19:00:00 -0000 In-Reply-To: References: <81113170-F590-4C51-BEA0-C8EF3A4422DA@contoso.com> <1501751620.3741.132.camel@klomp.org> <624331B2-472C-44AD-9034-05AD3CBBAE8A@cisco.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Evolution 3.22.6 (3.22.6-10.el7) Mime-Version: 1.0 X-IsSubscribed: yes X-SW-Source: 2017-q3/txt/msg00124.txt.bz2 Hi Djordje, On Thu, 2017-09-21 at 18:07 +0200, Djordje Todorovic wrote: > I have managed to integrate dwarflint on the top of elfutils-0.170 > release, so dwarflint and locstats can use all conveniences and > improvements of the newest libdw library (such as=C2=A0 > handling ops like DW_OP_GNU_entry_value). Very nice. > It is used internally by our compiler team. I have also managed to > add some new enhancements for locstats, such as reporting summary > either for formal parameter or local variable,=C2=A0 > reporting some kind of verbose info about particular variable or > formal parameter of particular function. > But, beside these enhancements that were interested for our compiler > team, dwarflint and locstats have a lot of good utilities that can be > really useful to compiler developers. For=C2=A0 > example, dwarflint tool can detect mistakes in debug sections such as > debug info, debug line etc., and help compiler developers to get rid > of those mistakes. Locstats can=C2=A0 > apparently report useful summary and show progress of making better > coverage for local variables or formal parameter in debug sections, > so it can save a lot of time to compiler=C2=A0 > developers. Yes, I know some compiler hackers have found it useful. And I was a little sad nobody had time to keep dwarflint up to date. It is really great to hear you have. > At least, maybe considering a making new branch called =E2=80=9Edwarflint= =E2=80=9C on > elfuitls git repository, which follows the newest source code, would > be good for the beginning, the same as it=C2=A0 > was on elfutils-0.152? Because, getting more feedback from users, > these tools would be better and more useful, so it can be, one day, > merged on to master branch. >=20 > If you are interested, I can share my patches. Yes please do share. I cannot promise to have a lot of time reviewing the code. But I am certainly interested. Creating a public "dwarflint" branch sounds like a good idea. Could you take a look at the CONTRIBUTING file? We don't have to be very formal for code not directly going onto master. But having at least a Signed-off-by line on the patches makes sure we can eventually integrate them officially. Thanks, Mark