Michael Elizabeth Chastain | 1 Sep 04:15 2003
Picon

Re: [patch] Remove hp300 dead files

Committed.

Michael C

2003-08-30  Michael Chastain  <mec <at> shout.net>

	* Makefile.in: Remove tm-hp300bsd.h, tm-hp300hpux.h.
	* config/m68k/nm-hp300hpux.h: Delete.
	* config/m68k/tm-hp300hpux.h: Delete.
	* config/m68k/xm-hp300hpux.h: Delete.
	* config/m68k/xm-hp300bsd.h: Delete.
	* config/djgpp/fnchange.lst: Remove nm-hp300hpux.h,
	tm-hp300hpux.h, xm-hp300hpux.h.
	* somsolib.c: Remove comment about hp300 shared libraries.

Paul Hilfinger | 1 Sep 11:39 2003

RFA: Changes to allow extensions to operator set


After a long hiatus, I am back.  In response to the last time I started
to submit Ada-related patches, Andrew Cagney wrote that it really
ought not be necessary to extend the set of operators in expression.h
to add Ada (or any other language, presumably).  OK, here is a set
of non-Ada-specific patches that allow one to add new language modules
without adding to the operators in expression.h.

While I was in here, I made a few other modifications that I hope meet
with general approval:

1. There was essentially duplicate code in prefixify_subexp and 
   length_of_subexp.  I have factored out the duplication.   [In the
   process, by the way, I happened to notice that BINOP_VAL was
   defined only in length_of_subexp, and indeed that there appears to
   be no code anywhere to handle its evaluation.]

2. Currently, the function dump_prefix_expression is used to dump 
   postfix expressions and dump_postfix_expression is used to dump
   prefix expressions.   It is probably a sign of the weakening
   effects of age on the intellect that I found this confusing, and
   have renamed dump_prefix_expression to dump_raw_expression (because
   in principle it doesn't really care whether the expression is 
   pre- or postfix), and dump_postfix_expression to dump_prefix_expression.

3. The current dump_prefix_expression tries to first print the expression 
   it is dumping with print_expression.  This doesn't work, however,
   because print_expression operates on prefix expressions, and
   the current dump_prefix_expression is only used on postfix 
   expressions.  I simply removed the print_expression.
(Continue reading)

Jerome Guitton | 1 Sep 17:06 2003
Picon

Re: [RFA] performance improvement of lookup_partial_symtab

Andrew Cagney (ac131313 <at> redhat.com):

> However, before that, wait for a comment from the symtab maintainers.

No reaction from the symtab maintainers so far, so I attached an updated patch,
with a more detailled comment.

Is it OK? I think it's a quite straight forward fix.

-- 
Jerome
2003-09-01  J. Guitton  <guitton <at> gnat.com>

	* symtab.c (lookup_partial_symbol): To improve the performance,
	try to avoid as much as possible to call source_full_path_of,
	by doing some preliminary testing.

Index: symtab.c
===================================================================
RCS file: /cvs/src/src/gdb/symtab.c,v
retrieving revision 1.114
diff -c -r1.114 symtab.c
*** symtab.c	8 Aug 2003 14:04:02 -0000	1.114
--- symtab.c	1 Sep 2003 14:54:12 -0000
***************
*** 267,274 ****
        }

(Continue reading)

Jerome Guitton | 1 Sep 17:16 2003
Picon

Ping: [RFA] ARM : prologue scan


No reaction about this patch... Any opinion from the ARM maintainers?
This patch should not interfere with Daniel Jacobowitz' work to modernize
the arm-tdep.c.

Jerome Guitton (guitton <at> act-europe.fr):

> 
> I have done some minor corrections on the previous patch, discard it.
> In attachment, the new version. No regression, no fixed.
> 
> -- 
> Jerome.

> 2003-07-21  J. Guitton  <guitton <at> gnat.com>
> 
> 	* arm-tdep.c (arm_skip_prologue): Add the handling of "sub ip, sp #n"
> 	and "add ip, sp #n", as these instructions can be found in a ATPCS
> 	compliant prologue.
> 	(arm_scan_prologue): Ditto.
> 
> Index: arm-tdep.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/arm-tdep.c,v
> retrieving revision 1.143
> diff -u -3 -p -r1.143 arm-tdep.c
> --- arm-tdep.c	13 Jun 2003 14:15:51 -0000	1.143
> +++ arm-tdep.c	22 Jul 2003 11:43:10 -0000
>  <at>  <at>  -449,6 +449,12  <at>  <at>  arm_skip_prologue (CORE_ADDR pc)
>        if (inst == 0xe1a0c00d)			/* mov ip, sp */
(Continue reading)

Andreas Schwab | 1 Sep 23:01 2003
Picon

Re: RFA: patch to ignore dump.exp for ia64

"J. Johnston" <jjohnstn <at> redhat.com> writes:

> Index: gdb.base/dump.exp
> ===================================================================
> RCS file: /cvs/src/src/gdb/testsuite/gdb.base/dump.exp,v
> retrieving revision 1.3
> diff -u -r1.3 dump.exp
> --- gdb.base/dump.exp	2 Jun 2003 15:51:24 -0000	1.3
> +++ gdb.base/dump.exp	28 Aug 2003 18:00:55 -0000
>  <at>  <at>  -33,12 +33,18  <at>  <at> 
>  set binfile  ${objdir}/${subdir}/${testfile}
>  set options  {debug}
>  
> +set is64bitonly "no"
> +

Why not just using 0/1 instead of "no"/"yes"?

Andreas.

--

-- 
Andreas Schwab, SuSE Labs, schwab <at> suse.de
SuSE Linux AG, Deutschherrnstr. 15-19, D-90429 N├╝rnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

David Carlton | 2 Sep 18:38 2003

Re: [rfa] generate symbols associated to namespaces

On Sun, 31 Aug 2003 15:29:50 -0400, Daniel Jacobowitz <drow <at> mvista.com> said:

> This patch needs a trivial update for the gdb.c++ -> gdb.cp rename,
> and IIRC one other merge update for the testsuite; and the comments
> we discussed downthread.  Mind reposting it?

I'll see if I can find time to do that, though I'm a bit busy right
now.  But, frankly, I do mind reposting it (though I'm certainly not
blaming you for this): the patch has been awaiting symtab review for
two and a half months, and I shouldn't have to spend the time keeping
it up to date while waiting for the symtab maintainers to get around
to it.

I had a lot of time to work on GDB through the end of June; I have
much less time to work on GDB now.  I'm still willing to spend the
time trying to get my branch merged into the mainline sources, but I'd
rather spend my limited GDB time doing things that are actually
productive (generating new patches, responding to criticism of them,
fixing bugs) instead of rehashing old patches.

David Carlton
carlton <at> kealia.com

J. Johnston | 2 Sep 20:34 2003
Picon

Re: RFA: patch to ignore dump.exp for ia64

Andreas Schwab wrote:
> "J. Johnston" <jjohnstn <at> redhat.com> writes:
> 
> 
>>Index: gdb.base/dump.exp
>>===================================================================
>>RCS file: /cvs/src/src/gdb/testsuite/gdb.base/dump.exp,v
>>retrieving revision 1.3
>>diff -u -r1.3 dump.exp
>>--- gdb.base/dump.exp	2 Jun 2003 15:51:24 -0000	1.3
>>+++ gdb.base/dump.exp	28 Aug 2003 18:00:55 -0000
>> <at>  <at>  -33,12 +33,18  <at>  <at> 
>> set binfile  ${objdir}/${subdir}/${testfile}
>> set options  {debug}
>> 
>>+set is64bitonly "no"
>>+
> 
> 
> Why not just using 0/1 instead of "no"/"yes"?
> 
> Andreas.
> 

Just a matter of arbitrary choice.  The lines I based it on were using strings.
It certainly isn't misleading or incorrect.

-- Jeff J.

(Continue reading)

Andrew Cagney | 2 Sep 20:46 2003
Picon

Re: Patch Gdb - target h8300 sim

Can you download a GDB 6.0 pre-release (GDB 5.3.91) from 
ftp://sources.redhat.com/pub/gdb/snapshots/branch/

many things h8300 related were fixed after GDB 5.3 was released.

Andrew

Jimi Xenidis | 2 Sep 20:57 2003
Picon

build break in RDA

Bug fix for:
   http://sources.redhat.com/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=gdb&pr=1343

ChangeLog:
2003-09-02  Jimi Xenidis  <jimix <at> watson.ibm.com>

	* linux-target.c (reginfo): The MQ register is for IBM POWER
	(pre-PowerPC) architecture, and is undefined in PPC GNU/Linux
	environment.

Index: ./rda/unix/linux-target.c
===================================================================
RCS file: /cvs/src/src/rda/unix/linux-target.c,v
retrieving revision 1.8
diff -u -r1.8 linux-target.c
--- ./rda/unix/linux-target.c	9 May 2003 22:06:12 -0000	1.8
+++ ./rda/unix/linux-target.c	2 Sep 2003 18:41:19 -0000
 <at>  <at>  -799,8 +799,10  <at>  <at> 
   { PT_CCR * 4,         4, GREGS,  PT_CCR * 4, 4, 4 },
   { PT_LNK * 4,         4, GREGS,  PT_LNK * 4, 4, 4 },
   { PT_CTR * 4,         4, GREGS,  PT_CTR * 4, 4, 4 },
-  { PT_XER * 4,         4, GREGS,  PT_XER * 4, 4, 4 },
-  { PT_MQ * 4,          4, GREGS,  PT_MQ * 4,  4, 4 }
+  { PT_XER * 4,         4, GREGS,  PT_XER * 4, 4, 4 }
+#ifdef PT_MQ
+  , { PT_MQ * 4,          4, GREGS,  PT_MQ * 4,  4, 4 }
+#endif
 };

 /* End of PPC_LINUX_TARGET */
(Continue reading)

Joel Brobecker | 2 Sep 22:19 2003

[RFC] decode_line_1 suggestion (problem with "list" command)

Hello,

We have a small problem with the list command. The following example
uses Ada, but I think the same kind of problem can occur in C++ as well.
The problem occurs when we use generics (templates in C++):

        generic
        function Gen1 return Integer;

        function Gen1 return Integer is
        begin
           return 0;
        end Gen1;

When we instantiate the generics more than once, vis:

        with Gen1;
        procedure Foo is
           function F1 is new Gen1;
           function F2 is new Gen1;

           X : Integer;
        begin
           X := F1;
        end Foo;

The compiler creates the code for each instantiation (which means
the code is duplicated). As a side effect, the line number information
is also duplicated.

(Continue reading)


Gmane