* ld.h (args_type): Add cref field.

	* lexsup.c (parse_args): Set command_line.cref.
	* ldmain.c (main): Check command_line.cref rather than
	link_info.notice_all.
	(notice): Likewise.

diff --git a/ld/ChangeLog b/ld/ChangeLog
index 581227002cb7f88c0316a07d50d5718aabf6dffd..23479d9afd2a108ae16734b8d6b20ce6e53398e2 100644 (file)
--- a/ld/ChangeLog
+++ b/ld/ChangeLog
@@ -1,3 +1,14 @@
+Tue Jul 30 14:46:42 1996  Ian Lance Taylor  <ian@cygnus.com>
+
+       * ld.h (args_type): Add cref field.
+       * lexsup.c (parse_args): Set command_line.cref.
+       * ldmain.c (main): Check command_line.cref rather than
+       link_info.notice_all.
+       (notice): Likewise.
+
+       * ldcref.c (output_one_cref): Don't crash if a symbol is defined
+       in a section without an owner.
+
 Mon Jul 29 17:23:33 1996  Ian Lance Taylor  <ian@cygnus.com>
 
        * fnmatch.h, fnmatch.c: New files.

diff --git a/ld/ld.h b/ld/ld.h
index c776de62784b3ef0a4ffa13ad62cbdbcd5456038..16c8096674df3231a3b9e8a890f4d16643fcbe43 100644 (file)
--- a/ld/ld.h
+++ b/ld/ld.h
@@ -84,6 +84,12 @@ typedef struct
   /* If true, build MIPS embedded PIC relocation tables in the output
      file.  */
   boolean embedded_relocs;
+
+  /* If true, force generation of a file with a .exe file. */
+  boolean force_exe_suffix;
+
+  /* If true, generate a cross reference report.  */
+  boolean cref;
 } args_type;
 
 extern args_type command_line;
@@ -145,4 +151,7 @@ extern int parsing_defsym;
 
 extern int yyparse PARAMS ((void));
 
+extern void add_cref PARAMS ((const char *, bfd *, asection *, bfd_vma));
+extern void output_cref PARAMS ((FILE *));
+
 #endif

diff --git a/ld/ldmain.c b/ld/ldmain.c
index f3c6d2a3e9a1e4a150bd98ae0110e15688992c02..13eb44875de5d6c644a0e3e920ff158f41994601 100644 (file)
--- a/ld/ldmain.c
+++ b/ld/ldmain.c
@@ -115,8 +115,8 @@ static boolean reloc_dangerous PARAMS ((struct bfd_link_info *, const char *,
 static boolean unattached_reloc PARAMS ((struct bfd_link_info *,
                                         const char *, bfd *, asection *,
                                         bfd_vma));
-static boolean notice_ysym PARAMS ((struct bfd_link_info *, const char *,
-                                   bfd *, asection *, bfd_vma));
+static boolean notice PARAMS ((struct bfd_link_info *, const char *,
+                              bfd *, asection *, bfd_vma));
 
 static struct bfd_link_callbacks link_callbacks =
 {
@@ -130,7 +130,7 @@ static struct bfd_link_callbacks link_callbacks =
   reloc_overflow,
   reloc_dangerous,
   unattached_reloc,
-  notice_ysym
+  notice
 };
 
 struct bfd_link_info link_info;
@@ -190,6 +190,7 @@ main (argc, argv)
   link_info.create_object_symbols_section = NULL;
   link_info.hash = NULL;
   link_info.keep_hash = NULL;
+  link_info.notice_all = false;
   link_info.notice_hash = NULL;
   link_info.wrap_hash = NULL;
   
@@ -320,6 +321,8 @@ main (argc, argv)
 
   if (config.map_file != NULL)
     lang_map ();
+  if (command_line.cref)
+    output_cref (config.map_file != NULL ? config.map_file : stdout);
 
   /* Even if we're producing relocateable output, some non-fatal errors should
      be reported in the exit status.  (What non-fatal errors, if any, do we
@@ -333,16 +336,57 @@ main (argc, argv)
                 output_filename);
        }
 
-      if (output_bfd->iostream)
-       fclose ((FILE *) (output_bfd->iostream));
+      /* The file will be removed by remove_output.  */
 
-      unlink (output_filename);
       xexit (1);
     }
   else
     {
       if (! bfd_close (output_bfd))
        einfo ("%F%B: final close failed: %E\n", output_bfd);
+
+      /* If the --force-exe-suffix is enabled, and we're making an
+        executable file and it doesn't end in .exe, copy it to one which does. */
+
+      if (! link_info.relocateable && command_line.force_exe_suffix)
+       {
+         int len = strlen (output_filename);
+         if (len < 4 
+             || (strcasecmp (output_filename + len - 4, ".exe") != 0
+                 && strcasecmp (output_filename + len - 4, ".dll") != 0))
+           {
+             FILE *src;
+             FILE *dst;
+             const int bsize = 4096;
+             char *buf = xmalloc (bsize);
+             int l;
+             char *dst_name = xmalloc (len + 5);
+             strcpy (dst_name, output_filename);
+             strcat (dst_name, ".exe");
+             src = fopen (output_filename, FOPEN_RB);
+             dst = fopen (dst_name, FOPEN_WB);
+
+             if (!src)
+               einfo ("%X%P: unable to open for source of copy `%s'\n", output_filename);
+             if (!dst)
+               einfo ("%X%P: unable to open for destination of copy `%s'\n", dst_name);
+             while ((l = fread (buf, 1, bsize, src)) > 0)
+               {
+                 int done = fwrite (buf, 1, l, dst);
+                 if (done != l)
+                   {
+                     einfo ("%P: Error writing file `%s'\n", dst_name);
+                   }
+               }
+             fclose (src);
+             if (!fclose (dst))
+               {
+                 einfo ("%P: Error closing file `%s'\n", dst_name);
+               }
+             free (dst_name);
+             free (buf);
+           }
+       }
     }
 
   END_PROGRESS (program_name);
@@ -632,10 +676,9 @@ add_archive_element (info, abfd, name)
 
   /* FIXME: The following fields are not set: header.next,
      header.type, closed, passive_position, symbol_count,
-     next_real_file, is_archive, target, real, common_section,
-     common_output_section, complained.  This bit of code is from the
-     old decode_library_subfile function.  I don't know whether any of
-     those fields matters.  */
+     next_real_file, is_archive, target, real.  This bit of code is
+     from the old decode_library_subfile function.  I don't know
+     whether any of those fields matters.  */
 
   ldlang_add_file (input);
 
@@ -1179,20 +1222,27 @@ unattached_reloc (info, name, abfd, section, address)
   return true;
 }
 
-/* This is called when a symbol in notice_hash is found.  Symbols are
-   put in notice_hash using the -y option.  */
+/* This is called if link_info.notice_all is set, or when a symbol in
+   link_info.notice_hash is found.  Symbols are put in notice_hash
+   using the -y option.  */
 
-/*ARGSUSED*/
 static boolean
-notice_ysym (info, name, abfd, section, value)
+notice (info, name, abfd, section, value)
      struct bfd_link_info *info;
      const char *name;
      bfd *abfd;
      asection *section;
      bfd_vma value;
 {
-  einfo ("%B: %s %s\n", abfd,
-        bfd_is_und_section (section) ? "reference to" : "definition of",
-        name);
+  if (! info->notice_all
+      || (info->notice_hash != NULL
+         && bfd_hash_lookup (info->notice_hash, name, false, false) != NULL))
+    einfo ("%B: %s %s\n", abfd,
+          bfd_is_und_section (section) ? "reference to" : "definition of",
+          name);
+
+  if (command_line.cref)
+    add_cref (name, abfd, section, value);
+
   return true;
 }