public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
From: Phil Muldoon <pmuldoon@redhat.com>
To: Eli Zaretskii <eliz@gnu.org>
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 09:17:00 -0000	[thread overview]
Message-ID: <53A3FC20.4030408@redhat.com> (raw)
In-Reply-To: <83ha3kvpv5.fsf@gnu.org>

On 16/06/14 16:25, Eli Zaretskii wrote:
>> Date: Mon, 16 Jun 2014 10:54:58 +0100
>> From: Phil Muldoon <pmuldoon@redhat.com>
>> CC: Tom Tromey <tromey@redhat.com>, "eliz@gnu.org" <eliz@gnu.org>

Thanks.

I've made the changes you have requested.  Tom plans to post the
whole patch-set again soon.  If required, I can just post the doc
changes here.  Otherwise, the changes will be part of the next
patch-set.

Comments in-line.

>> +@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 ...".

>> +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.  Some later plans yet to be implemented will
need the object code to stick around and execute at arbitrary points
(I.E., fast breakpoints with the object code being patched into the
inferior).  But those features do not exist yet.  When those features
become available I will submit a new doc patch describing when the
object code executes immediately, when it is deferred, and also
describe the life-cycle.


>> +When the language in @value{GDBN} is set to @samp{C}, the compiler will
>> +attempt to compile the source code with a @samp{C} compiler.  The source
>
> This begs the question: how will GDB know which compiler to invoke and
> by what name?

GDB invokes the GCC plugin which deals with this kind of housekeeping.
GDB loads libcc1.so and calls the exported functions in that library.
So from a GDB point of view, we don't care how the compiler is
invoked, or on the selection of the compiler (we'll, we do care, but
we trust in the GCC plugin to do the right thing).

>> +A compiler error would be raised as the variable @var{ff} no longer
>> +exists.
>
> This text hints, for the first time, that the compiled code is thrown
> away once it finishes execution.

I've written an explanation in the "compile command" section to warn
of the life-cycle of the object code, including the treatment of new
types and variables.

> Can the compile command access registers?

There's no definitive answer to this question.  "It depends" is the
best answer I can give. If the language the compile command is
compiling allows direct register access then yes, otherwise no.  We do
not prohibit the user doing anything in the language they could
normally do if they wrote the program with a text editor and compiled
it with GCC themselves.

Hope that helps

Cheers,

Phil

  reply	other threads:[~2014-06-20  9:17 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 10/14] make dwarf_expr_frame_base_1 public 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 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 01/14] introduce ui_file_write_for_put Tom Tromey
2014-05-16 15:26 ` [PATCH 08/14] introduce call_function_by_hand_dummy 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 [this message]
2014-06-20  9:42         ` Eli Zaretskii
2014-06-20 10:01           ` Phil Muldoon
2014-06-20 12:37             ` Eli Zaretskii
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 12/14] add linux_infcall_mmap Tom Tromey
2014-05-16 15:43 ` [PATCH 13/14] add s390_gcc_target_options 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=53A3FC20.4030408@redhat.com \
    --to=pmuldoon@redhat.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --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).