From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4374 invoked by alias); 26 Feb 2003 04:22:39 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 4219 invoked from network); 26 Feb 2003 04:22:37 -0000 Received: from unknown (HELO fep04-mail.bloor.is.net.cable.rogers.com) (66.185.86.74) by 172.16.49.205 with SMTP; 26 Feb 2003 04:22:37 -0000 Received: from tuliptree.org ([24.192.71.9]) by fep04-mail.bloor.is.net.cable.rogers.com (InterMail vM.5.01.05.06 201-253-122-126-106-20020509) with ESMTP id <20030226042235.QMEQ4278.fep04-mail.bloor.is.net.cable.rogers.com@tuliptree.org>; Tue, 25 Feb 2003 23:22:35 -0500 Message-ID: <3E5C412A.8070406@tuliptree.org> Date: Wed, 26 Feb 2003 04:22:00 -0000 From: Jim Wilson User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20021003 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Shrinivas Atre , gcc-patches@gcc.gnu.org, gdb-patches@gcc.gnu.org Subject: Re: [RFA/PATCH]: H8/300 - Update the generated debug information References: <69595093233BB547BB70CF5E492B63F2DA6751@sohm.kpit.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Authentication-Info: Submitted using SMTP AUTH PLAIN at fep04-mail.bloor.is.net.cable.rogers.com from [24.192.71.9] using ID at Tue, 25 Feb 2003 23:22:35 -0500 X-SW-Source: 2003-02/txt/msg00693.txt.bz2 > While generating COFF debug information for "int" type, the compiler > simply inserts type "T_INT" for this. > This doesn't say if the "int" is 32 bit or 16 bit. The SDB/coff debug format is old, limited, and unextensible. It is almost always wrong to use it. Since you are using gdb, a better solution is to use stabs which doesn't have this problem. Or, alternatively, if you must fix the SDB/coff debug info, I suggest doing the same thing Andrew Cagney suggested. Make gcc emit a special directive (.int32) and/or pass an option to gas. Have gas set an ELF section header flag when it sees the directive or the option. Then modify gdb to set the int size appropriately depending on the ELF section header flags. > * gcc/sdbout.c (plain_type_1): Update COFF debug output information. > Output "short int" if integer size is 16 bit. This isn't OK. It is wrong to emit a type of "short int" for an "int" even if they are the same size, because they aren't the same type. This will break the gdb testsuite. Jim