Re: [Linux-ia64] Re: gdb null ptr

[Kevin Buettner <kevinb@cygnus.com>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain, Size: 3743 bytes --]

[CC to linux-ia64@linuxia64.org trimmed since this is primarly a gdb
matter.]

On Nov 3, 10:20pm, Daniel Berlin wrote:

> > I will leave it to the dwarf2 maintainers to decide whether this
> > patch is acceptable or if it would be better to implement one of
> > Jim's other suggestions.
> > 
> 
> Unfortuantely , this is actually still wrong for languages other than
> C++, because we don't have the same guarantees about uniqueness in the name.
> 
> I was actually in the process of readying patches that add the same
> type of name based caching (based on mangled name) to partial and
> normal symbol reading, which gives us an amazing win for C++.
> 
> These patches also moved all of the name caching into "if (cu_language
> == language_cplus)" blocks, doing what we used to do in the old case
> (IE no caching).
> 
> Rather than let this stay broken until i finish cleaning up those
> patches, here is a patch that moves the type caching so it only
> happens for C++ CU's.

I hereby withdraw my patch from consideration in favor of Daniel's
patch.  In my opinion, Daniel's patch should go in ASAP since the code
in question has been broken since June 5 according to cvs annotate. 
For some reason, the corresponding ChangeLog entry is May 30.

The other alternative is to revert the May 30/June 5 patch.

> Unless other languages make the same guarantees, we can't do the same
> optimization.

Agreed.  I've constructed a small example and have convinced myself that
the code presently in dwarf2read.c is broken for C, but not for C++.

> I have added the same type of code kevin has to the patches i am
> readying.

Good.

You may wish to prepare a new interim patch which has a proper
ChangeLog entry, fixes the formatting of the long ``nameoftype = ...''
line, and which incorporates my proposed fix of not caching the type
unless it has a name or tag name.  This latter check is still
necessary to account for error conditions in reading the type die.  If
you feel that you do not have time to do this, let me know, and I'll
put it together for you.

Kevin
From inksupplies@kt.com.ar Mon Nov 06 18:39:00 2000
From: inksupplies <inksupplies@kt.com.ar>
To: gdb@sourceware.cygnus.com
Subject: Ahora imprima todo y gaste MENOS!!!
Date: Mon, 06 Nov 2000 18:39:00 -0000
Message-id: <e856f62b2cd17d16bf5908f8f627f5eb@NO-ID-FOUND.mhonarc.org>
X-SW-Source: 2000-11/msg00030.html
Content-length: 1348

Title: Si los cartuchos de su impresora a chorro de tinta







    Si los cartuchos de su impresora a chorro de
    tinta 
    Canon - Epson - Hewlett Packard - Lexmark
    
    le resultan caros... CLIQUEE AQUI !!!
    
    Informándonos la marca y modelo de su
    impresora 
    comenzará a ahorrar más de un 50% en sus
    costos de impresión
    NOTA: Su dirección nos
    fue proporcionada como interesada en recibir esta
    información, en caso de no ser así, rogamos disculpar la
    intromisión y responder este mensaje - Cliqueando en el
    siguiente link> Asunto: REMOVE [no recibirá nuevos mensajes] 
    Queremos comunicarnos con Ud.de la forma
    más saludable posible. 
    " NO MALGASTEMOS TANTO PAPEL EN
    PUBLICIDAD Y SE TALARAN MENOS ÁRBOLES" 
    !!! LA UTILIZACION DEL CORREO
    ELECTRÓNICO PRESERVA EL MEDIO AMBIENTE !!!
    Bajo
    el Decreto S.1618 TITULO III aprobado por el 105 Congreso
    base de las normativas internacionales sobre SPAM, esta carta
    no puede ser considerada SPAM mientras incluya una forma de
    ser removido y usted puede hacerlo ahora mismo respondiendo
    este e-mail con "remover" en el asunto. Gracias, y
    disculpe las molestias que pudiéramos haberle ocasionado en
    caso de que ésta información no sea de su interés.