From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32239 invoked by alias); 4 May 2016 01:03:09 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 32230 invoked by uid 89); 4 May 2016 01:03:09 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=pdr, our, highly X-HELO: mailapp01.imgtec.com Received: from mailapp01.imgtec.com (HELO mailapp01.imgtec.com) (195.59.15.196) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 04 May 2016 01:03:08 +0000 Received: from hhmail02.hh.imgtec.org (unknown [10.100.10.20]) by Websense Email with ESMTPS id 6E1DB852B25BE; Wed, 4 May 2016 02:03:04 +0100 (IST) Received: from [10.20.78.123] (10.20.78.123) by hhmail02.hh.imgtec.org (10.100.10.21) with Microsoft SMTP Server id 14.3.266.1; Wed, 4 May 2016 02:03:04 +0100 Date: Wed, 04 May 2016 01:03:00 -0000 From: "Maciej W. Rozycki" To: Trevor Saunders CC: Pedro Alves , Trevor Saunders , Yao Qi , Subject: Re: [PATCH, doc RFA] Remove support for "target m32rsdi" and "target mips/pmon/ddb/rockhopper/lsi" In-Reply-To: <20160503235813.GA4366@tsaunders-iceball.corp.tor1.mozilla.com> Message-ID: References: <1458230389-28233-1-git-send-email-palves@redhat.com> <8637rivld3.fsf@gmail.com> <56F13269.8020602@redhat.com> <56FD12FE.1030300@redhat.com> <70b62cd3-436b-2495-dd87-4bebf3de9e87@redhat.com> <20160503235813.GA4366@tsaunders-iceball.corp.tor1.mozilla.com> User-Agent: Alpine 2.00 (DEB 1167 2008-08-23) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" X-SW-Source: 2016-05/txt/msg00054.txt.bz2 On Wed, 4 May 2016, Trevor Saunders wrote: > > > Especially the heuristic > > > unwinder bits look highly suspicious to me, where the variable is updated > > > as the analysis proceeds. It could be that these variables can indeed go, > > > but perhaps something is missing that should be there. > > perhaps, my guess is that it was necessary at one time, but isn't > anymore. That can be tracked down rather easily now that we have GIT rather than CVS driving our repository, but I'm more concerned they might be symptoms of unfinished or buggy code. This is hairy stuff (by definition, actually -- where there's no DWARF information available we should try PDR records first, where available, before resorting, in desperation, to heuristics). Maciej