Re: new port for Xtensa processors (bfd files)

[Bob Wilson <bwilson at tensilica dot com>]

On Wednesday 02 April 2003 04:53 am, Nick Clifton wrote:
>   As I see it, Tensilica has three choices:
>
>    [ options 1 and 2 deleted]
>
>     3. Change the annotation on the files so that they are no longer
>        generated files but master source files.

I have no problem with this.  I've applied the following patches to change the 
comments about the files being automatically generated.

It would still be nice if changes to the interfaces are minimized and 
preferably done in consultation with Tensilica.  Due to constraints on our 
own release schedules, backward compatibility, etc. we may not always be able 
to change our code to match the master binutils files, at least not right 
away, and that could be an inconvenience to anyone trying to use binutils in 
combination with Tensilica's other tools.  I'd like to avoid those kinds 
problems as much as possible.  Honestly, I don't expect there to be much of a 
problem with any of this, but I thought it would be worth mentioning anyway.

BFD ChangeLog:

2003-04-02  Bob Wilson  <bob dot wilson at acm dot org>

        * xtensa-modules.c: Remove comment indicating that this is a
        generated file.

Include ChangeLog:

2003-04-02  Bob Wilson  <bob dot wilson at acm dot org>

        * xtensa-config.h: Remove comment indicating that this is a
        generated file.

LD ChangeLog:

2003-04-02  Bob Wilson  <bob dot wilson at acm dot org>

        * emulparams/xtensa-config.sh: Remove comment indicating that this is
        a generated file.


Index: xtensa-modules.c
===================================================================
RCS file: /cvs/src/src/bfd/xtensa-modules.c,v
retrieving revision 1.1
diff -c -3 -r1.1 xtensa-modules.c
*** xtensa-modules.c	1 Apr 2003 15:50:27 -0000	1.1
--- xtensa-modules.c	2 Apr 2003 16:46:38 -0000
***************
*** 17,24 ****
     along with this program; if not, write to the Free Software
     Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
  
- /* Automatically generated by gen-opcode-code - DO NOT EDIT! */
- 
  #include <xtensa-isa.h>
  #include "xtensa-isa-internal.h"
  #include "ansidecl.h"
--- 17,22 ----

Index: xtensa-config.h
===================================================================
RCS file: /cvs/src/src/include/xtensa-config.h,v
retrieving revision 1.1
diff -c -3 -r1.1 xtensa-config.h
*** xtensa-config.h	1 Apr 2003 15:50:30 -0000	1.1
--- xtensa-config.h	2 Apr 2003 16:54:08 -0000
***************
*** 2,26 ****
     Copyright (C) 2003 Free Software Foundation, Inc.
     Contributed by Bob Wilson (bwilson at tensilica dot com) at Tensilica.
  
! ** NOTE: This file was automatically generated by the Xtensa Processor
! ** Generator.  Changes made here will be lost when this file is
! ** updated or replaced with the settings for a different Xtensa
! ** processor configuration.  DO NOT EDIT!
  
! This program is free software; you can redistribute it and/or modify
! it under the terms of the GNU General Public License as published by
! the Free Software Foundation; either version 2, or (at your option)
! any later version.
  
! This program is distributed in the hope that it will be useful, but
! WITHOUT ANY WARRANTY; without even the implied warranty of
! MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
! General Public License for more details.
! 
! You should have received a copy of the GNU General Public License
! along with this program; if not, write to the Free Software
! Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
! */
  
  #ifndef XTENSA_CONFIG_H
  #define XTENSA_CONFIG_H
--- 2,20 ----
     Copyright (C) 2003 Free Software Foundation, Inc.
     Contributed by Bob Wilson (bwilson at tensilica dot com) at Tensilica.
  
!    This program is free software; you can redistribute it and/or modify
!    it under the terms of the GNU General Public License as published by
!    the Free Software Foundation; either version 2, or (at your option)
!    any later version.
  
!    This program is distributed in the hope that it will be useful, but
!    WITHOUT ANY WARRANTY; without even the implied warranty of
!    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
!    General Public License for more details.
  
!    You should have received a copy of the GNU General Public License
!    along with this program; if not, write to the Free Software
!    Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.  */
  
  #ifndef XTENSA_CONFIG_H
  #define XTENSA_CONFIG_H

Index: emulparams/xtensa-config.sh
===================================================================
RCS file: /cvs/src/src/ld/emulparams/xtensa-config.sh,v
retrieving revision 1.1
diff -c -3 -r1.1 xtensa-config.sh
*** emulparams/xtensa-config.sh	1 Apr 2003 15:50:31 -0000	1.1
--- emulparams/xtensa-config.sh	2 Apr 2003 16:55:50 -0000
***************
*** 1,8 ****
  # Xtensa configuration settings.
  
- ## NOTE: This file was automatically generated by the Xtensa Processor
- ## Generator.  Changes made here will be lost when this file is
- ## updated or replaced with the settings for a different Xtensa
- ## processor configuration.  DO NOT EDIT!
- 
  MAXPAGESIZE=0x1000
--- 1,3 ----