Message ID | 5ce1236e-6e83-0660-ddf6-3bc295a252e5@suse.cz |
---|---|
State | New |
Headers |
Return-Path: <gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org> X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id C2A48393BC3B for <patchwork@sourceware.org>; Wed, 12 Jan 2022 09:47:31 +0000 (GMT) X-Original-To: gcc-patches@gcc.gnu.org Delivered-To: gcc-patches@gcc.gnu.org Received: from smtp-out2.suse.de (smtp-out2.suse.de [195.135.220.29]) by sourceware.org (Postfix) with ESMTPS id D9CFD3858416 for <gcc-patches@gcc.gnu.org>; Wed, 12 Jan 2022 09:47:15 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org D9CFD3858416 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=suse.cz Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=suse.cz Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id C71531F385 for <gcc-patches@gcc.gnu.org>; Wed, 12 Jan 2022 09:47:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_rsa; t=1641980834; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=3UNRDnmV3lY8E7eXZ5DDyJLPkTLRwOZttGXCsIndpOE=; b=avGv1C3rDjp2jrLNH7GbQ84Wa5DpGn/HkxR5jitThNWBG8sd8CXNNImK7eFxlTpd7HGPfq xFOAd/98wl6cx7zQI1GFM1D5ntp92sdeY+1LuOOh9E03z9W9YHmjZ4xdxMjFrBMo4NH7bh w1IFcEe80fhSlGVRwvSW0xl+6CG6qOY= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.cz; s=susede2_ed25519; t=1641980834; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=3UNRDnmV3lY8E7eXZ5DDyJLPkTLRwOZttGXCsIndpOE=; b=w8VAANN4agTIYJ3eztbwuei2dTjPIi7Q/j1d572LX2fb+44ySI6hLfN8IOpH+mW3LvbjWG lamkI+OKmfFynlDw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id B8B5C13B3D for <gcc-patches@gcc.gnu.org>; Wed, 12 Jan 2022 09:47:14 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id 9SZjK6Kj3mFOVAAAMHmgww (envelope-from <mliska@suse.cz>) for <gcc-patches@gcc.gnu.org>; Wed, 12 Jan 2022 09:47:14 +0000 Message-ID: <5ce1236e-6e83-0660-ddf6-3bc295a252e5@suse.cz> Date: Wed, 12 Jan 2022 10:47:14 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.4.1 From: =?utf-8?q?Martin_Li=C5=A1ka?= <mliska@suse.cz> Subject: [PATCH] Fix redefined macro for epiphany. To: gcc-patches@gcc.gnu.org Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-11.7 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list <gcc-patches.gcc.gnu.org> List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=unsubscribe> List-Archive: <https://gcc.gnu.org/pipermail/gcc-patches/> List-Post: <mailto:gcc-patches@gcc.gnu.org> List-Help: <mailto:gcc-patches-request@gcc.gnu.org?subject=help> List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-patches>, <mailto:gcc-patches-request@gcc.gnu.org?subject=subscribe> Errors-To: gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org Sender: "Gcc-patches" <gcc-patches-bounces+patchwork=sourceware.org@gcc.gnu.org> |
Series |
Fix redefined macro for epiphany.
|
|
Commit Message
Martin Liška
Jan. 12, 2022, 9:47 a.m. UTC
The following warning is emitted gazillion times. Fixes: In file included from ./tm.h:23, from gcc/genconfig.c:25: gcc/config/elfos.h:209: warning: "READONLY_DATA_SECTION_ASM_OP" redefined 209 | #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" | In file included from ./tm.h:21, from gcc/genconfig.c:25: gcc/config/epiphany/epiphany.h:671: note: this is the location of the previous definition 671 | #define READONLY_DATA_SECTION_ASM_OP "\t.section .rodata" Ready to be installed? Thanks, Martin gcc/ChangeLog: * config/elfos.h (READONLY_DATA_SECTION_ASM_OP): Define only if not defined. --- gcc/config/elfos.h | 2 ++ 1 file changed, 2 insertions(+)
Comments
On Wed, Jan 12, 2022 at 10:47 AM Martin Liška <mliska@suse.cz> wrote: > > The following warning is emitted gazillion times. > > Fixes: > > In file included from ./tm.h:23, > from gcc/genconfig.c:25: > gcc/config/elfos.h:209: warning: "READONLY_DATA_SECTION_ASM_OP" redefined > 209 | #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" > | > In file included from ./tm.h:21, > from gcc/genconfig.c:25: > gcc/config/epiphany/epiphany.h:671: note: this is the location of the previous definition > 671 | #define READONLY_DATA_SECTION_ASM_OP "\t.section .rodata" > > Ready to be installed? Most other targets #undef READONLY_DATA_SECION_ASM_OP before defining, isn't the issue that the target doesn't do that and that it has uncommon order of tm_file in config.gcc? (${tm_file} first, before elfos.h) Richard. > Thanks, > Martin > > gcc/ChangeLog: > > * config/elfos.h (READONLY_DATA_SECTION_ASM_OP): Define only if > not defined. > --- > gcc/config/elfos.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/gcc/config/elfos.h b/gcc/config/elfos.h > index 2e0c709e585..add949ec940 100644 > --- a/gcc/config/elfos.h > +++ b/gcc/config/elfos.h > @@ -206,7 +206,9 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see > #define ASCII_DATA_ASM_OP "\t.ascii\t" > > /* Support a read-only data section. */ > +#ifndef READONLY_DATA_SECTION_ASM_OP > #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" > +#endif > > /* On svr4, we *do* have support for the .init and .fini sections, and we > can put stuff in there to be executed before and after `main'. We let > -- > 2.34.1 >
On 1/12/22 11:03, Richard Biener wrote:
> tm_file in config.gcc? (${tm_file} first, before elfos.h)
Heh, yes, that's the case. The following patch fixes that.
May I install it?
Cheers,
Martin
On Wed, Jan 12, 2022 at 11:10 AM Martin Liška <mliska@suse.cz> wrote: > > On 1/12/22 11:03, Richard Biener wrote: > > tm_file in config.gcc? (${tm_file} first, before elfos.h) > > Heh, yes, that's the case. The following patch fixes that. OK if the target builds with that. > May I install it? > Cheers, > Martin
On 1/12/2022 2:47 AM, Martin Liška wrote: > The following warning is emitted gazillion times. > > Fixes: > > In file included from ./tm.h:23, > from gcc/genconfig.c:25: > gcc/config/elfos.h:209: warning: "READONLY_DATA_SECTION_ASM_OP" redefined > 209 | #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" > | > In file included from ./tm.h:21, > from gcc/genconfig.c:25: > gcc/config/epiphany/epiphany.h:671: note: this is the location of the > previous definition > 671 | #define READONLY_DATA_SECTION_ASM_OP "\t.section .rodata" > > Ready to be installed? > Thanks, > Martin > > gcc/ChangeLog: > > * config/elfos.h (READONLY_DATA_SECTION_ASM_OP): Define only if > not defined. Please do. However, I think we should deprecate the epiphany. It's been broken for ~2 years randomly failing in reload. I tried to fix it once and quickly gave up. It currently ignores all testing failures in my tester as it's been horribly unstable. jeff
On Wed, Jan 12, 2022 at 3:23 PM Jeff Law via Gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > > > On 1/12/2022 2:47 AM, Martin Liška wrote: > > The following warning is emitted gazillion times. > > > > Fixes: > > > > In file included from ./tm.h:23, > > from gcc/genconfig.c:25: > > gcc/config/elfos.h:209: warning: "READONLY_DATA_SECTION_ASM_OP" redefined > > 209 | #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" > > | > > In file included from ./tm.h:21, > > from gcc/genconfig.c:25: > > gcc/config/epiphany/epiphany.h:671: note: this is the location of the > > previous definition > > 671 | #define READONLY_DATA_SECTION_ASM_OP "\t.section .rodata" > > > > Ready to be installed? > > Thanks, > > Martin > > > > gcc/ChangeLog: > > > > * config/elfos.h (READONLY_DATA_SECTION_ASM_OP): Define only if > > not defined. > Please do. > > However, I think we should deprecate the epiphany. It's been broken for > ~2 years randomly failing in reload. I tried to fix it once and quickly > gave up. It currently ignores all testing failures in my tester as it's > been horribly unstable. Joern is still listed as maintainer so let's CC him before doing anything. That said, if it builds it's good enough for GCC 12 - we can discuss any target deprecations during stage1 (or deprecate those that don't even build right now, though I'm not aware of any). Richard. > > jeff
On 1/12/2022 7:56 AM, Richard Biener wrote: > On Wed, Jan 12, 2022 at 3:23 PM Jeff Law via Gcc-patches > <gcc-patches@gcc.gnu.org> wrote: >> >> >> On 1/12/2022 2:47 AM, Martin Liška wrote: >>> The following warning is emitted gazillion times. >>> >>> Fixes: >>> >>> In file included from ./tm.h:23, >>> from gcc/genconfig.c:25: >>> gcc/config/elfos.h:209: warning: "READONLY_DATA_SECTION_ASM_OP" redefined >>> 209 | #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" >>> | >>> In file included from ./tm.h:21, >>> from gcc/genconfig.c:25: >>> gcc/config/epiphany/epiphany.h:671: note: this is the location of the >>> previous definition >>> 671 | #define READONLY_DATA_SECTION_ASM_OP "\t.section .rodata" >>> >>> Ready to be installed? >>> Thanks, >>> Martin >>> >>> gcc/ChangeLog: >>> >>> * config/elfos.h (READONLY_DATA_SECTION_ASM_OP): Define only if >>> not defined. >> Please do. >> >> However, I think we should deprecate the epiphany. It's been broken for >> ~2 years randomly failing in reload. I tried to fix it once and quickly >> gave up. It currently ignores all testing failures in my tester as it's >> been horribly unstable. > Joern is still listed as maintainer so let's CC him before doing > anything. That said, > if it builds it's good enough for GCC 12 - we can discuss any target > deprecations > during stage1 (or deprecate those that don't even build right now, > though I'm not > aware of any). cr16 still uses cc0. It's marked as deprecated in gcc-12 and assuming it doesn't get fixed, removal in gcc-13. m32c should be deprecated. While it'll build libgcc, it'll fault (due to register allocation/reloading issues) building newlib or anything relatively complex. This has been the case for at least 2 years. epiphany will build newlib consistently, so I guess keeping it another release is OK to see if Joern wants to step up and fix its reload issues. Jeff
diff --git a/gcc/config/elfos.h b/gcc/config/elfos.h index 2e0c709e585..add949ec940 100644 --- a/gcc/config/elfos.h +++ b/gcc/config/elfos.h @@ -206,7 +206,9 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see #define ASCII_DATA_ASM_OP "\t.ascii\t" /* Support a read-only data section. */ +#ifndef READONLY_DATA_SECTION_ASM_OP #define READONLY_DATA_SECTION_ASM_OP "\t.section\t.rodata" +#endif /* On svr4, we *do* have support for the .init and .fini sections, and we can put stuff in there to be executed before and after `main'. We let