public inbox for elfutils@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Djordje Todorovic <djordje.todorovic@rt-rk.com>
Cc: "Ananthakrishna Sowda (asowda)" <asowda@cisco.com>,
	"elfutils-devel@sourceware.org" <elfutils-devel@sourceware.org>,
	"nemanja.popov@rt-rk.com" <nemanja.popov@rt-rk.com>,
	petar.jovanovic@rt-rk.com, Nikola Prica <nikola.prica@rt-rk.com>
Subject: Re: Latest version of dwarflint?
Date: Fri, 29 Sep 2017 12:08:00 -0000	[thread overview]
Message-ID: <20170929115843.GA2482@stream> (raw)
In-Reply-To: <c2b5f92f-0798-1e43-2095-b282bd5d7879@rt-rk.com>

Hi Djordje,

On Thu, Sep 28, 2017 at 03:36:14PM +0200, Djordje Todorovic wrote:
> Please find attached compressed folder with dwarflint patches. I’m sending it in this way because I think changes are not that a small to be sent in a mail body or with git mail.

Thanks. It looks like the mailinglist didn't like the big zip file.
So for now I have put it here:
https://gnu.wildebeest.org/~mark/dwarflint_patches.zip

> All of the patches can be applied on to elfutils-0.170 release. I have run
> dwarflint test suites and all of the tests are PASS at this point.

I haven't gotten them all applied yet.
I am adjusting the Makefile.in changes to the Makefile.am files,
so we don't have generated files in git. And since I am on RHEL7
with an older g++ I had to add some autoconf magic to get C++11 support:
https://www.gnu.org/software/autoconf-archive/ax_cxx_compile_stdcxx.html

As soon as I have it cleaned up to build I'll add the patches to a
new dwarflint branch.

> Besides bug fixes and code updates, please note a new option added to
> locstats tool (e.g. --sub-category=formal_parameters), which reports summary
> either for formal parameters or local variables.
> 
> Also, I’m working on adding a new option which can preset debug info about
> particular variable of particular function in more “human readable” way. At
> this point, that option can handle only C functions and I’m customizing it
> also to handle methods of C++ classes as well. There can be output more
> additional info, e.g. compilation unit etc. I think it would be very useful
> to users and when I’m done with it, I’ll share it.
> 
> Running the tool on GDB executable with option that I have mentioned looks as following:
> 
> eu-locstats  --show=amd64_epilogue_frame_cache[this_frame] gdb
> Function: amd64_epilogue_frame_cache
> Formal parameter: this_frame
> Low PC: 0x40b4e0
> High PC relative: 0x156
> Range of the function: [0x40b4e0, 0x40b636)
> Location list of the formal parameter:
> 	PC range: [40b4e0,40b504) expression: {DW_OP_reg5}
> 	PC range: [40b504,40b519) expression: {DW_OP_reg6}
> 	PC range: [40b519,40b538) expression: {DW_OP_GNU_entry_value(1) {DW_OP_reg5}, DW_OP_stack_value}
> 
> So, for example, if compiler developers adding a handling of a new DWARF operation, in quick way they can see if there is appropriate information generated.
> 
> Thanks,
> Djordje

  parent reply	other threads:[~2017-09-29 12:08 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-02 19:47 Ananthakrishna Sowda (asowda)
2017-08-03  9:13 ` Mark Wielaard
2017-09-21 15:59   ` Ananthakrishna Sowda (asowda)
2017-09-21 16:07     ` Djordje Todorovic
2017-09-21 19:00       ` Mark Wielaard
2017-09-22 13:37         ` Djordje Todorovic
2017-09-25 16:18           ` Mark Wielaard
2017-09-26 11:39             ` Djordje Todorovic
     [not found]             ` <c2b5f92f-0798-1e43-2095-b282bd5d7879@rt-rk.com>
2017-09-29 12:08               ` Mark Wielaard [this message]
2017-09-29 12:26                 ` Mark Wielaard
2017-09-29 12:44                   ` Djordje Todorovic
2017-09-29 13:07                     ` Mark Wielaard
2017-10-06 19:53                 ` Mark Wielaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20170929115843.GA2482@stream \
    --to=mark@klomp.org \
    --cc=asowda@cisco.com \
    --cc=djordje.todorovic@rt-rk.com \
    --cc=elfutils-devel@sourceware.org \
    --cc=nemanja.popov@rt-rk.com \
    --cc=nikola.prica@rt-rk.com \
    --cc=petar.jovanovic@rt-rk.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).