From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8688 invoked by alias); 17 Dec 2002 19:25:00 -0000 Mailing-List: contact cgen-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cgen-owner@sources.redhat.com Received: (qmail 8669 invoked from network); 17 Dec 2002 19:24:59 -0000 Received: from unknown (HELO neon-gw.transmeta.com) (63.209.4.196) by 209.249.29.67 with SMTP; 17 Dec 2002 19:24:59 -0000 Received: (from root@localhost) by neon-gw.transmeta.com (8.9.3/8.9.3) id LAA20950; Tue, 17 Dec 2002 11:24:32 -0800 Received: from mailhost.transmeta.com(10.1.1.15) by neon-gw.transmeta.com via smap (V2.1) id xma020938; Tue, 17 Dec 02 11:24:04 -0800 Received: from casey.transmeta.com (casey.transmeta.com [10.10.25.22]) by deepthought.transmeta.com (8.11.6/8.11.6) with ESMTP id gBHJO8R10300; Tue, 17 Dec 2002 11:24:08 -0800 (PST) Received: (from dje@localhost) by casey.transmeta.com (8.9.3/8.7.3) id LAA04237; Tue, 17 Dec 2002 11:24:08 -0800 From: Doug Evans MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <15871.31192.305439.813418@casey.transmeta.com> Date: Tue, 17 Dec 2002 11:25:00 -0000 To: DJ Delorie Cc: amacleod@redhat.com, amodra@bigpond.net.au, binutils@sources.redhat.com, cgen@sources.redhat.com Subject: Re: New Sanyo Stormy16 relocations In-Reply-To: <200212170353.gBH3r9f14238@envy.delorie.com> References: <1039041358.28757.307.camel@p4> <20021204225643.GS27956@bubble.sa.bigpond.net.au> <1039043233.28767.313.camel@p4> <200212170353.gBH3r9f14238@envy.delorie.com> X-SW-Source: 2002-q4/txt/msg00097.txt.bz2 DJ Delorie writes: > I'm committing this approved patch on Andrew's behalf. The cgen parts > were approved by FChE off-list. > > > bfd/ChangeLog > > * elf32-xstormy16.c (xstormy16_elf_howto): Add R_XSTORMY16_LO16 > > and R_XSTORMY16_HI16) howto entries. > > (xstormy16_reloc_map): Map R_XSTORMY16_{LO,HI}16 to BFD_RELOC_{LO,HI}16. > > (xstormy16_info_to_howto_rela): Use R_XSTORMY16_GNU_VTINHERIT to > > determine the start of the second reloc table. > > > > cgen/ChangeLog > > * cpu/xstormy16.cpu (imm16): Call handler immediate16. > > * cpu/xstormy16.opc (parse_small_immediate): Return on '@'. > > (parse_immediate16): Handle immediate16 values, which now include > > @hi(label) and @lo(label) > > > > gas/ChangeLog > > * config/tc-xstormy16.c (md_cgen_lookup_reloc): If a relocation > > has already been set up, use it. > > > > include/ChangeLog > > * elf/xstormy16.h (START_RELOC_NUMBERS) Add relocation numbers > > for R_XSTORMY16_LO16 and R_XSTORMY16_HI16. > > > > opcodes/ChangeLog > > * opcodes/xstormy16-asm.c: Regenerate. Having to get cgen approval for cpu-specific changes sucks. People should be able to police their own ports. gcc port maintainers don't have to get approval for changes to their ports. I don't understand why this would be any different. Is there a reason for this (anal-retentive) procedure? [I'm not suggesting you or anyone else is actually imposing this of course. Maybe people just think that's the way things are.] But, if approval is required, methinks binutils is a better place to provide approval for .opc changes (e.g. complaints about warnings :-).