From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6696 invoked by alias); 4 May 2002 03:13:48 -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 6688 invoked from network); 4 May 2002 03:13:47 -0000 Received: from unknown (HELO zwingli.cygnus.com) (208.245.165.35) by sources.redhat.com with SMTP; 4 May 2002 03:13:47 -0000 Received: by zwingli.cygnus.com (Postfix, from userid 442) id 9820D5EA11; Fri, 3 May 2002 22:13:45 -0500 (EST) To: Daniel Jacobowitz Cc: gdb-patches@sources.redhat.com Subject: Re: RFA: Check for corruption of cv_type chain References: <20020503215423.B21045EA11@zwingli.cygnus.com> <20020503221418.GA29293@nevyn.them.org> <20020504002507.GA4915@nevyn.them.org> From: Jim Blandy Date: Fri, 03 May 2002 20:13:00 -0000 In-Reply-To: <20020504002507.GA4915@nevyn.them.org> Message-ID: User-Agent: Gnus/5.09 (Gnus v5.9.0) Emacs/21.1 MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-SW-Source: 2002-05/txt/msg00099.txt.bz2 Daniel Jacobowitz writes: > On Fri, May 03, 2002 at 07:10:21PM -0500, Jim Blandy wrote: > > > > Daniel Jacobowitz writes: > > > On Fri, May 03, 2002 at 04:54:23PM -0500, Jim Blandy wrote: > > > > > > > ! # These tests are intended to catch problems in the STABS reader, so I > > > > ! # think it's okay to override the preference the user expressed in > > > > ! # their $DEJAGNU file or whatever, and explicitly request extended > > > > ! # stabs debugging information. But, if we don't know how to do that, > > > > ! # these tests still should pass just fine if the executable uses some > > > > ! # other format. > > > > ! # > > > > ! # Plain old -gstabs isn't enough, by the way. See the notes for > > > > ! # "doesn't corrupt cv_type chain", below. > > > > ! if {$gcc_compiled} { > > > > ! set debug_option additional_flags=-gstabs+ > > > > ! } else { > > > > ! set debug_option debug > > > > ! } > > > > ! set options [list c++ $debug_option] > > > > > > Please don't... -gstabs+ is a configuration; if we test that > > > configuration, which is the default pre-3.1, then we will see the bug. > > > If I run a DWARF-2 testsuite explicitly I don't really want to look at > > > bugs in the stabs reader. > > > > It doesn't seem to the default pre-3.1. -gstabs is. (I think.) > > No, -gstabs+ is definitely the default. Our C++ debugging with -gstabs > is practically nil, so we'd notice :) [goes and checks his compilers] I see what's going on now. In GCC 2.96, -g doesn't get you `const' and `volatile' qualifiers in stabs --- that's why I thought -gstabs was the default. But in fact, GCC 2.96 won't give you those qualifiers no matter how nicely you ask. The current GCC sources default to Dwarf 2. So except for the window between the point where we taught dbxout.c to emit the qualifiers, and the point where we made Dwarf 2 the default (a pretty narrow window, in terms of installed base, no?), it doesn't matter whether -gstabs+ is the default or not; this test won't do its job with the default flags. Oh well. If someone is explicitly interested in -gstabs+, they should run the test suite with that flag, and then the test will do its job. I guess I shouldn't worry about it any more.