This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
RE: [PATCH v2] Bug 17394: we cannot put a break-point at a global function for ASM file
- From: "mihail dot nistor at freescale dot com" <mihail dot nistor at freescale dot com>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: Keith Seitz <keiths at redhat dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Sun, 21 Dec 2014 09:13:17 +0000
- Subject: RE: [PATCH v2] Bug 17394: we cannot put a break-point at a global function for ASM file
- Authentication-results: sourceware.org; auth=none
- Authentication-results: spf=none (sender IP is ) smtp dot mailfrom=mihail dot nistor at freescale dot com;
- References: <1410954786-17690-1-git-send-email-mihail dot nistor at freescale dot com> <542459B9 dot 9090705 at redhat dot com> <23ca9e13576840e280392b69e5cc7de8 at BN1PR0301MB0628 dot namprd03 dot prod dot outlook dot com> <7dec43bc41e044fcbba3289ca7cb2697 at BN1PR0301MB0628 dot namprd03 dot prod dot outlook dot com> <542B0854 dot 6050804 at redhat dot com> <20141213130429 dot GA12185 at adacore dot com> <BN1PR0301MB062886DD142FCD57BFD3FA92E26F0 at BN1PR0301MB0628 dot namprd03 dot prod dot outlook dot com> <20141220163307 dot GE12884 at adacore dot com>
Hello Joel,
Thank you very much for your help and guidance.
In the future I will do as you told me and I hope everything will go smoothly.
Thank you again,
Mihai
-----Original Message-----
From: Joel Brobecker [mailto:brobecker@adacore.com]
Sent: Saturday, December 20, 2014 6:33 PM
To: Nistor Mihail-MNISTOR1
Cc: Keith Seitz; gdb-patches@sourceware.org
Subject: Re: [PATCH v2] Bug 17394: we cannot put a break-point at a global function for ASM file
On Mon, Dec 15, 2014 at 08:47:19PM +0000, mihail.nistor@freescale.com wrote:
> Thank you very much for your support.
>
> You will find enclosed a new patch.
[...]
> > gdb/ChangeLog:
> > 2014-09-30 Keith Seitz <keiths@redhat.com>
> > Mihail-Marian Nistor <mihail.nistor@freescale.com>
> >
> > PR gdb/17394
> > * linespec.c (struct collect_minsyms): Add new member `symtab'.
> > (add_minsym): Handle cases where info.symtab is non-NULL.
> > (search_minsyms_for_name): Add new parameter `symtab'.
> > Handle limiting searches to a specific symtab.
> > (add_matching_symtabs_to_info): Search through minimal symbols
> > for language_asm files for which no new symbols are found.
> >
> > gdb/testsuite/ChangeLog:
> > 2014-09-30 Mihail-Marian Nistor <mihail.nistor@freescale.com>
> >
> > PR gdb/17394
> > * gdb.linespec/break-asm-file.c: New file.
> > * gdb.linespec/break-asm-file.exp: New file.
> > * gdb.linespec/break-asm-file0.s: New file.
> > * gdb.linespec/break-asm-file1.s: New file.
Thank you. The patch is approved.
I have a few small additional remarks that came up following the
review:
The most important one that I almost tripped over, is the fact that the patch no longer builds on today's master due to a change in struct symtab. This goes to show that, when one asks for the patch to be rebased, we should just go ahead and do not just that, but also re-test it, even if the file that it touches hasn't changed in the interim. I've made the required changes and re-done testing this time around.
In the future, it would really help if you could send the commits to us, rather than just the diff. What we're interesting in, in addition to the diff, is the commit's revision log. It should contain a detailed description of the problem you're trying to solve, and how you're solving it (usually, we try to put the "why"
in the code). We try to do that at every iteration of the review, so we can keep an eye on the contents of the revision log and make sure it's complete and accurate. And another advantage is that it allows me to push your patch in under 10 seconds rather than having to apply a diff, make sure I don't miss some changes along the way, commit, write the revision log, get your email address (as commit author), etc.
Since there is a PR and testcase, and this patch has been delayed quite a bit for lack of review, I will take care of creating the commit this time.
Attached is the commit I just pushed.
--
Joel