From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 95687 invoked by alias); 7 May 2015 23:57:45 -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 95670 invoked by uid 89); 7 May 2015 23:57:44 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-Spam-User: qpsmtpd, 2 recipients X-HELO: bes.se.axis.com Received: from bes.se.axis.com (HELO bes.se.axis.com) (195.60.68.10) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 07 May 2015 23:57:43 +0000 Received: from localhost (localhost [127.0.0.1]) by bes.se.axis.com (Postfix) with ESMTP id 89D702E44F; Fri, 8 May 2015 01:57:40 +0200 (CEST) Received: from bes.se.axis.com ([IPv6:::ffff:127.0.0.1]) by localhost (bes.se.axis.com [::ffff:127.0.0.1]) (amavisd-new, port 10024) with LMTP id 4voupqEhx88l; Fri, 8 May 2015 01:57:40 +0200 (CEST) Received: from boulder.se.axis.com (boulder.se.axis.com [10.0.2.104]) by bes.se.axis.com (Postfix) with ESMTP id B5A892E442; Fri, 8 May 2015 01:57:39 +0200 (CEST) Received: from boulder.se.axis.com (localhost [127.0.0.1]) by postfix.imss71 (Postfix) with ESMTP id 943961065; Fri, 8 May 2015 01:57:39 +0200 (CEST) Received: from thoth.se.axis.com (thoth.se.axis.com [10.0.2.173]) by boulder.se.axis.com (Postfix) with ESMTP id 88A001042; Fri, 8 May 2015 01:57:39 +0200 (CEST) Received: from ignucius.se.axis.com (ignucius.se.axis.com [10.88.21.50]) by thoth.se.axis.com (Postfix) with ESMTP id 8514334005; Fri, 8 May 2015 01:57:39 +0200 (CEST) Received: from ignucius.se.axis.com (localhost [127.0.0.1]) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) with ESMTP id t47NvdBc030201; Fri, 8 May 2015 01:57:39 +0200 Received: (from hp@localhost) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) id t47Nvcr2030197; Fri, 8 May 2015 01:57:38 +0200 Date: Thu, 07 May 2015 23:57:00 -0000 Message-Id: <201505072357.t47Nvcr2030197@ignucius.se.axis.com> From: Hans-Peter Nilsson To: vapier@gentoo.org CC: nickc@redhat.com, gdb-patches@sourceware.org, binutils@sourceware.org In-reply-to: <20150507063614.GZ30924@vapier> (message from Mike Frysinger on Thu, 7 May 2015 08:36:14 +0200) Subject: Re: Fix sim fallout from arm assembler complaining about symbols named as insns MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT X-SW-Source: 2015-05/txt/msg00173.txt.bz2 > From: Mike Frysinger > Date: Thu, 7 May 2015 08:36:14 +0200 > so people can't have a global variable named "bl" now ? or "ldr" ? that > doesn't seem like the right direction in which case this patch isn't really > needed ... > -mike Yeah, as you've noticed I haven't applied it yet. I reconsidered after VP noting that there actually *are* gcc test-suite cases failing because of this. Sorry Nick, but I'd like to see a -mwarn-syms being passed by default by gcc, before believing this warning-by-default behavior is here to stay. The cure (helping a rare programming error with an unexpected failure mode by means of a sledgehammer fix) is IMHO worse than the disease. Add a target hook to the SYM = EXPR gas code and check there? (But don't forget to check that the symbol doesn't yet exist or at least isn't global.) brgds, H-P