From: Jeff Law <law@redhat.com>
To: Pierre-Marie de Rodat <derodat@adacore.com>, gcc-patches@gcc.gnu.org
Subject: Re: [PATCH 0/2] Python testcases to check DWARF output
Date: Thu, 03 Aug 2017 16:13:00 -0000 [thread overview]
Message-ID: <ac1e52a6-ad92-5191-534e-4817afb20625@redhat.com> (raw)
In-Reply-To: <ffb57aa5-8368-43ed-9634-367fbd428229@adacore.com>
On 08/03/2017 02:27 AM, Pierre-Marie de Rodat wrote:
> On 08/02/2017 05:43 PM, Jeff Law wrote:
>> I hate to throw in a wrench at this point, but has anyone looked at
>> dwgrep from Petr Machata? He's not doing much with it anymore, but it
>> might provide enough of a dwarf scanning framework to be useful for
>> testing purposes.
>
> Sure, no problem: I first started talking publicly about this one week
> ago, so itâs definitely not too late to mention alternatives. ;-) I
> learned about dwgrep two years ago and forgot about it, so thank you for
> the idea. I started to have a look at it, and for now I donât think itâs
> a good match in this context:
>
> 1. itâs an ELF only tool;
> 2. it must be built, requiring external dependencies: cmake and
> elfutils;
> 3. in order to use it, one must learn a dedicated post-fix language
> (Zwerg)
>
> For 1. I think this is a true problem, as it means for instance that we
> could not test DWARF on Windows and Darwin setups. Unless we add PE and
> Mach-O handling in dwgrep of course, but that does not sound easy and
> will bring other external dependencies.
>
> For 3. I feel that, for someone who is comfortable with Python, it will
> be easier to deal with a Python library (the dwarfutils in my patch)
> than having to learn yet another DSL. I think thatâs precisely why some
> people would like to have a Python test framework rather than a TCL one.
> Working with a âusualâ imperative language looks easier than with
> postfix expressions. Smaller cognitive load.
>
> Actually I see another problem: pattern will have to vary depending on
> the target platform (for instance 32/64bit or depending on the DWARF
> version). Of course we could duplicate whole patterns in testcases to
> take this into account, but thatâs like code duplication: I think we
> should be able to include small âX if 32bit else Yâ in patterns, and I
> donât think we can do that with Zwerg (no way to pass something like
> environment variables).
>
> Of course, I have written a âcompetitorâ tool: I guess my judgment is
> biased. :-) So other opinions are welcome!
Thanks. I just wanted to raise it as a possibility. It looks like it's
not a good fit here, so let's not let it be a distraction.
jeff
prev parent reply other threads:[~2017-08-03 16:13 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-26 16:01 Pierre-Marie de Rodat
2017-07-26 16:01 ` [PATCH 2/2] Introduce " Pierre-Marie de Rodat
2017-07-26 17:10 ` David Malcolm
2017-07-27 8:59 ` Pierre-Marie de Rodat
2017-07-27 8:36 ` Richard Biener
2017-07-27 10:09 ` Pierre-Marie de Rodat
2017-07-26 16:01 ` [PATCH 1/2] Introduce testsuite support to run Python tests Pierre-Marie de Rodat
2017-07-26 16:25 ` David Malcolm
2017-07-26 16:35 ` Pierre-Marie de Rodat
2017-07-26 16:48 ` David Malcolm
2017-07-27 8:49 ` Pierre-Marie de Rodat
2017-07-27 13:40 ` David Malcolm
2017-08-02 18:43 ` Jeff Law
2017-08-03 8:27 ` Pierre-Marie de Rodat
2017-07-27 8:50 ` Matthias Klose
2017-07-27 10:09 ` Pierre-Marie de Rodat
2017-07-26 16:16 ` [PATCH 0/2] Python testcases to check DWARF output David Malcolm
2017-07-26 16:26 ` Pierre-Marie de Rodat
2017-07-26 21:25 ` Mike Stump
2017-07-27 7:52 ` Richard Biener
2017-07-27 9:09 ` Pierre-Marie de Rodat
2017-08-03 22:23 ` Mike Stump
2017-08-06 14:35 ` Iain Buclaw
2017-08-02 15:44 ` Jeff Law
2017-08-02 15:43 ` Jeff Law
2017-08-03 8:27 ` Pierre-Marie de Rodat
2017-08-03 16:13 ` Jeff Law [this message]
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=ac1e52a6-ad92-5191-534e-4817afb20625@redhat.com \
--to=law@redhat.com \
--cc=derodat@adacore.com \
--cc=gcc-patches@gcc.gnu.org \
/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).