Re: [RFA] New port: CR16: BFD Changes required by the gdb port

[Pedro Alves <palves@redhat.com>]

On 01/08/2013 11:24 AM, Kaushik Phatak wrote:
> Hi Nick,
> Thanks for applying these changes with appropriate Changelogs.
> 
> I noticed that the entries for cr16 were missing in the latest gdb snapshot
> in configure.ac and configure.
> 
>   cr16-*-*)
>     noconfigdirs="$noconfigdirs gdb"
>     ;;
> This entry is present in the 7.5 release version of gdb but not
> in the latest snapshot, gdb-7.5.50.20130108
> 
> How is this controlled, in gcc or here? Could you please throw some light on this.

On gdb's git mirror, I see:

commit ce3c5eda0b8d664790bc633a1013193a3289dff0
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Sun Aug 26 14:34:38 2012 +0000

    Sync toplevel files with GCC tree

...
-  cr16-*-*)
-    noconfigdirs="$noconfigdirs gdb"
-    ;;
   d10v-*-*)
     noconfigdirs="$noconfigdirs target-libgloss"
     ;;
@@ -924,7 +939,8 @@ case "${target}" in
     noconfigdirs="$noconfigdirs libgui itcl ld"
     ;;
   ia64*-*-*vms*)
-    noconfigdirs="$noconfigdirs libgui itcl"
+    # No ld support yet.
+    noconfigdirs="$noconfigdirs libgui itcl ld"
     ;;
   i[[3456789]]86-w64-mingw*)
     ;;
@@ -1069,16 +1085,13 @@ case "${host}" in
   *-mingw*)
     host_makefile_frag="config/mh-mingw"
     ;;
-  *-interix*)
-    host_makefile_frag="config/mh-interix"
-    ;;
   hppa*-hp-hpux10*)
     host_makefile_frag="config/mh-pa-hpux10"
     ;;
   hppa*-hp-hpux*)
     host_makefile_frag="config/mh-pa"
     ;;
-  hppa*-*)
+  hppa*-*)
     host_makefile_frag="config/mh-pa"
     ;;

Dunno why these were removed with that merge.  The
ChangeLog entries brought in don't seen to
mention these changes.  Maybe it was a merge error
where local binutils changes were lost?

> Please let me know if I need to submit any patch for this as couple of hunks
> from my gdb patch are failing due to this.

Please always base the work you submit against the current
development mainline.  For GDB, that is:

 http://sourceware.org/gdb/current/

-- 
Pedro Alves