public inbox for gdb@sourceware.org
 help / color / mirror / Atom feed
From: Pierre-Marie de Rodat <derodat@adacore.com>
To: Joel Brobecker <brobecker@adacore.com>,
	Ulrich Weigand <uweigand@de.ibm.com>
Cc: Sivanupandi Pitchumani <Pitchumani.Sivanupandi@atmel.com>,
	"gdb@sourceware.org" <gdb@sourceware.org>,
	Andrew Burgess <andrew.burgess@embecosm.com>,
	"tom@tromey.com" <tom@tromey.com>
Subject: Re: dynamic array's upper bound evaluated as address for AVR target
Date: Fri, 23 Oct 2015 14:19:00 -0000	[thread overview]
Message-ID: <562A41EA.6020304@adacore.com> (raw)
In-Reply-To: <561E5A7C.1040902@adacore.com>

On 10/14/2015 09:37 AM, Pierre-Marie de Rodat wrote:
> And to me as well: thank you Ulrich for this crystal clear explanation.
> :-) I will give it a try next week, then.

Quick update: I started to dig into this. I still have a hard time 
wrapping the current implementation in my mind[1] and thinking about 
what we should do exactly:

   * Have different types for expressions and location lists (hence 
potentially renaming existing code): looks like a big commit!

   * Introduce another entry point to evaluate dynamic properties as 
expressions. Looks like a small change but a weaker design: how to 
evaluate depends a property depends on the property, not the evaluation 
context, so it would be great to keep types distinct.

I cannot make any commitment on a schedule to implement this, though.


[1] Especially considering that dynamic properties handle only locations 
that resolve to an inferior address. Location resolution for variables 
don’t rely on this mechanism, which is some kind of inconsistency. On 
the other hand, dynamic properties are DWARF-specific…

-- 
Pierre-Marie de Rodat

  reply	other threads:[~2015-10-23 14:19 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-13  9:08 Sivanupandi, Pitchumani
2015-10-13 14:44 ` Pierre-Marie de Rodat
2015-10-14  6:33   ` Sivanupandi, Pitchumani
2015-10-14  6:54     ` Pierre-Marie de Rodat
2015-10-14  7:54       ` Sivanupandi, Pitchumani
2015-10-14  8:26         ` Pierre-Marie de Rodat
2015-10-14  9:41           ` Ulrich Weigand
2015-10-14 12:26             ` Joel Brobecker
2015-10-14 13:37               ` Pierre-Marie de Rodat
2015-10-23 14:19                 ` Pierre-Marie de Rodat [this message]
2015-10-23 16:47                   ` Ulrich Weigand
2015-10-14 10:15           ` Sivanupandi, Pitchumani
2015-10-14 13:39             ` Pierre-Marie de Rodat

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=562A41EA.6020304@adacore.com \
    --to=derodat@adacore.com \
    --cc=Pitchumani.Sivanupandi@atmel.com \
    --cc=andrew.burgess@embecosm.com \
    --cc=brobecker@adacore.com \
    --cc=gdb@sourceware.org \
    --cc=tom@tromey.com \
    --cc=uweigand@de.ibm.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).