c-opts.c (c_common_handle_option): Handle -fpch-preprocess.
authorGeoffrey Keating <geoffk@apple.com>
Tue, 22 Jun 2004 06:51:56 +0000 (06:51 +0000)
committerGeoffrey Keating <geoffk@gcc.gnu.org>
Tue, 22 Jun 2004 06:51:56 +0000 (06:51 +0000)
2004-06-21  Geoffrey Keating  <geoffk@apple.com>

* c-opts.c (c_common_handle_option): Handle -fpch-preprocess.
* c-common.h (flag_pch_preprocess): Declare.
(c_common_pch_pragma): Likewise.
* c-common.c (flag_pch_preprocess): New.
* c-pch.c (c_common_read_pch): Support -fpreprocess-only.
(c_common_pch_pragma): New.
* c-ppoutput.c (cb_read_pch): New.
(init_pp_output): Support -fpch-preprocess.
* c-pragma.c (init_pragma): Support #pragma GNUC pch_preprocess.
* c.opt (fpch-preprocess): New.
* gcc.c (cpp_options): When save-temps, pass -fpch-preprocess.
* doc/cppopts.texi: Document -fpch-preprocess.
* doc/invoke.texi (Precompiled Headers): Mention that
-fpreprocessed is safe for PCH.  Mention that if an option is
listed as safe that doesn't mean it does what you expect.

Index: gcc/testsuite/ChangeLog
2004-06-21  Geoffrey Keating  <geoffk@apple.com>

* gcc.dg/pch/save-temps-1.c: New file.
* gcc.dg/pch/save-temps-1.hs: New file.

Index: libcpp/ChangeLog
2004-06-21  Geoffrey Keating  <geoffk@apple.com>

* files.c (should_stack_file): Correct swapped parameters to call
to cb.read_pch.
* pch.c (cpp_valid_state): Handle -fpreprocessed.

From-SVN: r83478

17 files changed:
gcc/ChangeLog
gcc/c-common.c
gcc/c-common.h
gcc/c-opts.c
gcc/c-pch.c
gcc/c-ppoutput.c
gcc/c-pragma.c
gcc/c.opt
gcc/doc/cppopts.texi
gcc/doc/invoke.texi
gcc/gcc.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.dg/pch/save-temps-1.c [new file with mode: 0644]
gcc/testsuite/gcc.dg/pch/save-temps-1.hs [new file with mode: 0644]
libcpp/ChangeLog
libcpp/files.c
libcpp/pch.c

index b2659c10c290cf6970abf306a5040635ca36ef8f..88a9d9575cb88e5f4fb5a719edf5e07dfab4f904 100644 (file)
@@ -1,3 +1,21 @@
+2004-06-21  Geoffrey Keating  <geoffk@apple.com>
+
+       * c-opts.c (c_common_handle_option): Handle -fpch-preprocess.
+       * c-common.h (flag_pch_preprocess): Declare.
+       (c_common_pch_pragma): Likewise.
+       * c-common.c (flag_pch_preprocess): New.
+       * c-pch.c (c_common_read_pch): Support -fpreprocess-only.
+       (c_common_pch_pragma): New.
+       * c-ppoutput.c (cb_read_pch): New.
+       (init_pp_output): Support -fpch-preprocess.
+       * c-pragma.c (init_pragma): Support #pragma GNUC pch_preprocess.
+       * c.opt (fpch-preprocess): New.
+       * gcc.c (cpp_options): When save-temps, pass -fpch-preprocess.
+       * doc/cppopts.texi: Document -fpch-preprocess.
+       * doc/invoke.texi (Precompiled Headers): Mention that
+       -fpreprocessed is safe for PCH.  Mention that if an option is
+       listed as safe that doesn't mean it does what you expect.
+
 2004-06-22  Ben Elliston  <bje@au.ibm.com>
 
        * tree-ssa.c (ssa_redirect_edge): Correct leading comment.
index 79d9d9ac85970e9e68c60471fec9aa950d0b3aac..ab5b2ae3b499d9e2d903102872216dcd78610b9d 100644 (file)
@@ -217,6 +217,10 @@ char flag_dump_macros;
 
 char flag_dump_includes;
 
+/* Nonzero means process PCH files while preprocessing.  */
+
+bool flag_pch_preprocess;
+
 /* The file name to which we should write a precompiled header, or
    NULL if no header will be written in this compile.  */
 
index bd0a5b36a0e11f53195c8db479f357d6409c7799..03c6c75e1ea170639548766833089a59b736017a 100644 (file)
@@ -350,6 +350,10 @@ extern char flag_dump_macros;
 
 extern char flag_dump_includes;
 
+/* Nonzero means process PCH files while preprocessing.  */
+
+extern bool flag_pch_preprocess;
+
 /* The file name to which we should write a precompiled header, or
    NULL if no header will be written in this compile.  */
 
@@ -1096,12 +1100,15 @@ extern void c_genericize (tree);
 extern int c_gimplify_expr (tree *, tree *, tree *);
 extern tree c_build_bind_expr (tree, tree);
 
+/* In c-pch.c  */
 extern void pch_init (void);
 extern int c_common_valid_pch (cpp_reader *pfile, const char *name, int fd);
 extern void c_common_read_pch (cpp_reader *pfile, const char *name, int fd,
                               const char *orig);
 extern void c_common_write_pch (void);
 extern void c_common_no_more_pch (void);
+extern void c_common_pch_pragma (cpp_reader *pfile);
+
 extern void builtin_define_with_value (const char *, const char *, int);
 extern void c_stddef_cpp_builtins (void);
 extern void fe_file_change (const struct line_map *);
index 8025cd7897cfff5f0ed9682c00e0165ec3385c62..2ae9c13364527620a2282320b1844ea3c470976f 100644 (file)
@@ -879,6 +879,10 @@ c_common_handle_option (size_t scode, const char *arg, int value)
       cpp_opts->restore_pch_deps = value;
       break;
 
+    case OPT_fpch_preprocess:
+      flag_pch_preprocess = value;
+      break;
+
     case OPT_fpermissive:
       flag_permissive = value;
       break;
index 03cf3bf08bcae4379cedfd1c4fb2ddb65e8e5a90..eb043bd089eaca213c14afcf70fcb4d9f7779902 100644 (file)
@@ -393,8 +393,6 @@ c_common_read_pch (cpp_reader *pfile, const char *name,
 {
   FILE *f;
   struct c_pch_header h;
-  char *buf;
-  unsigned long written;
   struct save_macro_data *smd;
   
   f = fdopen (fd, "rb");
@@ -412,18 +410,30 @@ c_common_read_pch (cpp_reader *pfile, const char *name,
       return;
     }
 
-  buf = xmalloc (16384);
-  for (written = 0; written < h.asm_size; )
+  if (!flag_preprocess_only)
     {
-      long size = h.asm_size - written;
-      if (size > 16384)
-       size = 16384;
-      if (fread (buf, size, 1, f) != 1
-         || fwrite (buf, size, 1, asm_out_file) != 1)
-       cpp_errno (pfile, CPP_DL_ERROR, "reading");
-      written += size;
+      unsigned long written;
+      char * buf = xmalloc (16384);
+
+      for (written = 0; written < h.asm_size; )
+       {
+         long size = h.asm_size - written;
+         if (size > 16384)
+           size = 16384;
+         if (fread (buf, size, 1, f) != 1
+             || fwrite (buf, size, 1, asm_out_file) != 1)
+           cpp_errno (pfile, CPP_DL_ERROR, "reading");
+         written += size;
+       }
+      free (buf);
+    }
+  else
+    {
+      /* If we're preprocessing, don't write to a NULL
+        asm_out_file.  */
+      if (fseek (f, h.asm_size, SEEK_CUR) != 0)
+       cpp_errno (pfile, CPP_DL_ERROR, "seeking");
     }
-  free (buf);
 
   cpp_prepare_state (pfile, &smd);
 
@@ -446,3 +456,47 @@ c_common_no_more_pch (void)
       host_hooks.gt_pch_use_address (NULL, 0, -1, 0);
     }
 }
+
+/* Handle #pragma GCC pch_preprocess, to load in the PCH file.  */
+
+#ifndef O_BINARY
+# define O_BINARY 0
+#endif
+
+void
+c_common_pch_pragma (cpp_reader *pfile)
+{
+  tree name_t;
+  const char *name;
+  int fd;
+
+  if (c_lex (&name_t) != CPP_STRING)
+    {
+      error ("malformed #pragma GCC pch_preprocess, ignored");
+      return;
+    }
+
+  if (! cpp_get_options (pfile)->preprocessed)
+    {
+      error ("pch_preprocess pragma should only be used with -fpreprocessed");
+      inform ("use #include instead");
+      return;
+    }
+
+  name = TREE_STRING_POINTER (name_t);
+  
+  fd = open (name, O_RDONLY | O_BINARY, 0666);
+  if (fd == -1)
+    fatal_error ("%s: couldn't open PCH file: %m\n", name);
+  
+  if (c_common_valid_pch (pfile, name, fd) != 1)
+    {
+      if (!cpp_get_options (pfile)->warn_invalid_pch)
+       inform ("use -Winvalid-pch for more information");
+      fatal_error ("%s: PCH file was invalid", name);
+    }
+  
+  c_common_read_pch (pfile, name, fd, name);
+  
+  close (fd);
+}
index ef6d8e6d4f3a4a354cd7e4b5315aff376575f2af..06816e9b885e2face6234b6f7a5b1c210eb3a179 100644 (file)
@@ -57,6 +57,8 @@ static void cb_include (cpp_reader *, fileline, const unsigned char *,
                        const char *, int);
 static void cb_ident (cpp_reader *, fileline, const cpp_string *);
 static void cb_def_pragma (cpp_reader *, fileline);
+static void cb_read_pch (cpp_reader *pfile, const char *name,
+                        int fd, const char *orig_name);
 
 /* Preprocess and output.  */
 void
@@ -106,6 +108,12 @@ init_pp_output (FILE *out_stream)
   if (flag_dump_includes)
     cb->include  = cb_include;
 
+  if (flag_pch_preprocess)
+    {
+      cb->valid_pch = c_common_valid_pch;
+      cb->read_pch = cb_read_pch;
+    }
+
   if (flag_dump_macros == 'N' || flag_dump_macros == 'D')
     {
       cb->define = cb_define;
@@ -416,3 +424,17 @@ dump_macro (cpp_reader *pfile, cpp_hashnode *node, void *v ATTRIBUTE_UNUSED)
 
   return 1;
 }
+
+/* Load in the PCH file NAME, open on FD.  It was originally searched for
+   by ORIG_NAME.  Also, print out a #include command so that the PCH
+   file can be loaded when the preprocessed output is compiled.  */
+
+static void
+cb_read_pch (cpp_reader *pfile, const char *name,
+            int fd, const char *orig_name ATTRIBUTE_UNUSED)
+{
+  c_common_read_pch (pfile, name, fd, orig_name);
+  
+  fprintf (print.outf, "#pragma GCC pch_preprocess \"%s\"\n", name);
+  print.src_line++;
+}
index 93836ea8652ac2b4a88d32973a40747121fec395..ba59fb3a589a1ae67ed7f65381590f66932c5aec 100644 (file)
@@ -589,6 +589,8 @@ init_pragma (void)
   c_register_pragma (0, "redefine_extname", handle_pragma_redefine_extname);
   c_register_pragma (0, "extern_prefix", handle_pragma_extern_prefix);
 
+  c_register_pragma ("GCC", "pch_preprocess", c_common_pch_pragma);
+
 #ifdef REGISTER_TARGET_PRAGMAS
   REGISTER_TARGET_PRAGMAS ();
 #endif
index f162803009024b9ec6db5792283aaeda9cdbf928..8898efdfd3e096d3cec4a91f0292b20e1b3174f0 100644 (file)
--- a/gcc/c.opt
+++ b/gcc/c.opt
@@ -599,6 +599,10 @@ Enable optional diagnostics
 fpch-deps
 C ObjC C++ ObjC++
 
+fpch-preprocess
+C ObjC C++ ObjC++
+Look for and use PCH files even when preprocessing
+
 fpermissive
 C++ ObjC++
 Downgrade conformance errors to warnings
index 468f17f13fc8299e58d8536e2d43329d5284847b..1b6307b2215d526db24390b60e3ab930f9ad2a04 100644 (file)
@@ -323,6 +323,24 @@ precompiled header would be listed and not the files that were used to
 create it because those files are not consulted when a precompiled
 header is used.
 
+@item -fpch-preprocess
+@opindex fpch-preprocess
+This option allows use of a precompiled header (@pxref{Precompiled
+Headers}) together with @option{-E}.  It inserts a special @code{#pragma},
+@code{#pragma GCC pch_preprocess "<filename>"} in the output to mark
+the place where the precompiled header was found, and its filename.  When
+@code{-fpreprocessed} is in use, GCC recognizes this @code{#pragma} and
+loads the PCH.
+
+This option is off by default, because the resulting preprocessed output
+is only really suitable as input to GCC.  It is switched on by
+@option{-save-temps}.
+
+You should not write this @code{#pragma} in your own code, but it is
+safe to edit the filename if the PCH file is available in a different
+location.  The filename may be absolute or it may be relative to GCC's
+current directory.
+
 @end ifclear
 @item -x c
 @itemx -x c++
index 2dd2b8239d754c0c075fe16a6a8c391a989eeedc..fcfb521812f31a50d5ea795d591f3ae2911e7647 100644 (file)
@@ -11809,7 +11809,7 @@ which options are safe to change and which are not; the safest choice
 is to use exactly the same options when generating and using the
 precompiled header.  The following are known to be safe:
 
-@gccoptlist{-pedantic-errors}
+@gccoptlist{-fpreprocessed -pedantic-errors}
 
 @end itemize
 
@@ -11819,6 +11819,12 @@ find an option combination that doesn't work and doesn't cause the
 precompiled header to be ignored, please consider filing a bug report,
 see @ref{Bugs}.
 
+If you do use differing options when generating and using the
+precompiled header, the actual behaviour will be a mixture of the
+behaviour for the options.  For instance, if you use @option{-g} to
+generate the precompiled header but not when using it, you may or may
+not get debugging information for routines in the precompiled header.
+
 @node Running Protoize
 @section Running Protoize
 
index e377dc37923ce9015cae548a356c18c571bf0ce9..677d9e74c37138969c5127fe4e414dafbeb8d7d9 100644 (file)
--- a/gcc/gcc.c
+++ b/gcc/gcc.c
@@ -783,7 +783,8 @@ static const char *cpp_unique_options =
    in turn cause preprocessor symbols to be defined specially.  */
 static const char *cpp_options =
 "%(cpp_unique_options) %1 %{m*} %{std*} %{ansi} %{W*&pedantic*} %{w} %{f*}\
- %{g*:%{!g0:%{!fno-working-directory:-fworking-directory}}} %{O*} %{undef}";
+ %{g*:%{!g0:%{!fno-working-directory:-fworking-directory}}} %{O*} %{undef}\
+ %{save-temps:-fpch-preprocess}";
 
 /* This contains cpp options which are not passed when the preprocessor
    output will be used by another program.  */
index 73cebd14a476ce4335cd33f4e437d4622c5fb49d..cbc1a4b8448150a5b9648ff6f3ffeb9de53ab9fc 100644 (file)
@@ -1,3 +1,8 @@
+2004-06-21  Geoffrey Keating  <geoffk@apple.com>
+
+       * gcc.dg/pch/save-temps-1.c: New file.
+       * gcc.dg/pch/save-temps-1.hs: New file.
+
 2004-06-22  Janne Blomqvist  <jblomqvi@cc.hut.fi>
 
        * gfortran.fortran-torture/execute/iolength_1.f90: New test.
diff --git a/gcc/testsuite/gcc.dg/pch/save-temps-1.c b/gcc/testsuite/gcc.dg/pch/save-temps-1.c
new file mode 100644 (file)
index 0000000..87cedb3
--- /dev/null
@@ -0,0 +1,7 @@
+/* { dg-options "-I. -save-temps" } */
+#include "save-temps-1.h"
+#ifndef T
+#error T not defined
+#endif
+#include <stddef.h>
+int x;
diff --git a/gcc/testsuite/gcc.dg/pch/save-temps-1.hs b/gcc/testsuite/gcc.dg/pch/save-temps-1.hs
new file mode 100644 (file)
index 0000000..2fe5a3e
--- /dev/null
@@ -0,0 +1,5 @@
+#define T 123
+int foo (void) 
+{
+  return 3;
+}
index 236c0685e4db9a725042f1acafe545412f317fb8..897d4ac0b268f05672ce0d68c513c45ec5bf80a9 100644 (file)
@@ -1,3 +1,9 @@
+2004-06-21  Geoffrey Keating  <geoffk@apple.com>
+
+       * files.c (should_stack_file): Correct swapped parameters to call
+       to cb.read_pch.
+       * pch.c (cpp_valid_state): Handle -fpreprocessed.
+
 2004-06-15  Paolo Bonzini  <bonzini@gnu.org>
 
        * Makefile.in: Regenerate with automake 1.8.5.
index 6a18d71a896631d32376d568a34f1b0f14a76a94..068065279baa15b1c0ef34f40968da818b0655bb 100644 (file)
@@ -625,7 +625,7 @@ should_stack_file (cpp_reader *pfile, _cpp_file *file, bool import)
   /* Handle PCH files immediately; don't stack them.  */
   if (file->pch)
     {
-      pfile->cb.read_pch (pfile, file->path, file->fd, file->pchname);
+      pfile->cb.read_pch (pfile, file->pchname, file->fd, file->path);
       close (file->fd);
       file->fd = -1;
       return false;
index a9d139a30b76121e0ed457b791c20853c6b95567..a343cd857f01ddff22ed1512bfe8182a0493a051 100644 (file)
@@ -438,13 +438,22 @@ cpp_valid_state (cpp_reader *r, const char *name, int fd)
       if (m.name_length == 0)
        break;
 
+      /* If this file is already preprocessed, there won't be any
+        macros defined, and that's OK.  */
+      if (CPP_OPTION (r, preprocessed))
+       {
+         if (lseek (fd, m.definition_length, SEEK_CUR) == -1)
+           goto error;
+         continue;
+       }
+
       if (m.definition_length > namebufsz)
        {
          free (namebuf);
          namebufsz = m.definition_length + 256;
          namebuf = xmalloc (namebufsz);
        }
-      
+
       if ((size_t)read (fd, namebuf, m.definition_length) 
          != m.definition_length)
        goto error;