public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Phil Muldoon <pmuldoon@redhat.com>
Cc: gdb-patches@sourceware.org, tromey@redhat.com
Subject: Re: (Doc ping [for news and manual]) --  [PATCH 14/14] the "compile" command
Date: Fri, 20 Jun 2014 12:37:00 -0000	[thread overview]
Message-ID: <8361jvsqot.fsf@gnu.org> (raw)
In-Reply-To: <53A40662.60708@redhat.com>

> Date: Fri, 20 Jun 2014 11:01:06 +0100
> From: Phil Muldoon <pmuldoon@redhat.com>
> CC: gdb-patches@sourceware.org, tromey@redhat.com
> 
> On 20/06/14 10:42, Eli Zaretskii wrote:
> >> Date: Fri, 20 Jun 2014 10:17:20 +0100
> >> From: Phil Muldoon <pmuldoon@redhat.com>
> >> CC: gdb-patches@sourceware.org, tromey@redhat.com
> >>
> >>>> +@value{GDBN}, or the compiler does not support this feature
> >>>
> >>> I think it would be good to say here which compiler(s) in what
> >>> version(s) started supporting this feature.
> >>
> >> We actually don't have one yet.  That will change soon. The GCC
> >> changes are being reviewed now (this project is a cross GCC/GDB
> >> project).  Once there is a released version number associated with a
> >> GCC version, I will add a "since GCC version ...".
> >
> > Aren't there plans in place to make this part of a known GCC version,
> > or maybe a branch where these changes are being made is already slated
> > to be included in some known version?  If so, please state that
> > version; it can be changed later if plans change or life intervenes.
> >
> > But even saying it's a GCC feature is already much more than we tell
> > now.
> 
> No, how can there be plans to slot it into a version when it is under
> review? I would suspect "the next version that GCC releases" but I am
> unfamiliar with how GCC does its release versioning.  Minor/Major and
> so on.  I don't reject your idea, I just don't know how to give a
> version when there is no version yet.

How about asking the GCC people?

Anyway, if there's no way to guesstimate the version, let's at least
say this feature requires GCC with the <put the name here> plugin.

> >>>> +compiles and links successfully, @value{GDBN} will load the object-code
> >>>> +emitted, and execute it within the context of the currently selected
> >>>> +inferior.
> >>>
> >>> When you say "and execute it", you don't mean right away, yes?
> >>> Because that's what the text conveys.  Will the execution commence
> >>> immediately, or only when the program counter gets to this code?
> >>
> >> Yes right away.  The object code is loaded and placed in a dummy frame
> >> and executed immediately.
> >
> > Then I guess "compile" is a misleading name.
> 
> Why? Is compiling not happening?  What other command names do you
> suggest?

'compile-and-execute', for example.  Or maybe there should be no new
command, but rather an extension of 'print' and 'call', which can
already call functions in the inferior.

"Compile" means just compile to object code.  When I "compile" a
program, it doesn't run.  By contrast, here it does.

  reply	other threads:[~2014-06-20 12:37 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-16 15:26 [PATCH 00/14] let gdb reuse gcc's C compiler Tom Tromey
2014-05-16 15:26 ` [PATCH 03/14] add some missing ops to DWARF assembler Tom Tromey
2014-05-16 15:26 ` [PATCH 07/14] add infcall_mmap and gcc_target_options gdbarch methods Tom Tromey
2014-05-19  6:13   ` Yao Qi
2014-05-19  6:41     ` Jan Kratochvil
2014-05-19  6:47       ` Jan Kratochvil
2014-05-19  7:42       ` Yao Qi
2014-05-19  8:26         ` Agovic, Sanimir
2014-05-19 11:15           ` Jan Kratochvil
2014-05-16 15:26 ` [PATCH 08/14] introduce call_function_by_hand_dummy Tom Tromey
2014-05-16 15:26 ` [PATCH 01/14] introduce ui_file_write_for_put Tom Tromey
2014-05-16 15:26 ` [PATCH 02/14] add gcc/gdb interface files Tom Tromey
2014-05-19  8:06   ` Yao Qi
2014-05-19 13:53     ` Tom Tromey
2014-05-16 15:26 ` [PATCH 10/14] make dwarf_expr_frame_base_1 public Tom Tromey
2014-05-16 15:27 ` [PATCH 14/14] the "compile" command Tom Tromey
2014-06-16  9:55   ` (Doc ping [for news and manual]) -- " Phil Muldoon
2014-06-16 15:25     ` Eli Zaretskii
2014-06-20  9:17       ` Phil Muldoon
2014-06-20  9:42         ` Eli Zaretskii
2014-06-20 10:01           ` Phil Muldoon
2014-06-20 12:37             ` Eli Zaretskii [this message]
2014-06-20 14:42             ` Tom Tromey
2014-06-20 19:00               ` Eli Zaretskii
2014-06-21  7:32                 ` Eli Zaretskii
2014-05-16 15:41 ` [PATCH 05/14] change how the CLI handles comments Tom Tromey
2014-05-18 23:26   ` Doug Evans
2014-05-19 14:07     ` Phil Muldoon
2014-06-03  7:04       ` Doug Evans
2014-05-16 15:43 ` [PATCH 13/14] add s390_gcc_target_options Tom Tromey
2014-05-16 15:43 ` [PATCH 12/14] add linux_infcall_mmap Tom Tromey
2014-05-16 15:43 ` [PATCH 09/14] split dwarf2_fetch_cfa_info from dwarf2_compile_expr_to_ax Tom Tromey
2014-05-16 15:56 ` [PATCH 04/14] add make_unqualified_type Tom Tromey
2014-05-16 16:18 ` [PATCH 06/14] add dummy frame destructor Tom Tromey
2014-05-16 16:18 ` [PATCH 11/14] export dwarf2_reg_to_regnum_or_error Tom Tromey

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=8361jvsqot.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=pmuldoon@redhat.com \
    --cc=tromey@redhat.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).