[pph] Add FIXME comment to avoid finalizing decls when generating pph image.

gcc/passes.c

 /* Top level of GCC compilers (cc1, cc1plus, etc.)
    Copyright (C) 1987, 1988, 1989, 1992, 1993, 1994, 1995, 1996, 1997, 1998,
    1999, 2000, 2001, 2002, 2003, 2004, 2005, 2006, 2007, 2008, 2009, 2010
    Free Software Foundation, Inc.
 
 This file is part of GCC.
 
 GCC 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 3, or (at your option) any later
@@ skipping 166 matching lines @@
 
          We do output all variables and rely on
          callgraph code to defer them except for forward declarations
          (see gcc.c-torture/compile/920624-1.c) */
       if ((at_end
            || !DECL_DEFER_OUTPUT (decl)
            || DECL_INITIAL (decl))
           && !DECL_EXTERNAL (decl))
         {
           /* When reading LTO unit, we also read varpool, so do not
-             rebuild it.  */
+             rebuild it.
+             FIXME pph: This is also true for pph and we should not
+             call varpool_finalize_decl when generating a pph image.  */
           if (in_lto_p && !at_end)
             ;
           else if (TREE_CODE (decl) != FUNCTION_DECL)
             varpool_finalize_decl (decl);
         }
 
 #ifdef ASM_FINISH_DECLARE_OBJECT
       if (decl == last_assemble_variable_decl)
         {
           ASM_FINISH_DECLARE_OBJECT (asm_out_file, decl,
@@ skipping 2353 matching lines @@
     }
   if (dump_file && e)
     {
       fprintf (dump_file, "Already processed call to:\n");
       dump_cgraph_node (dump_file, e->caller);
     }
   return e != NULL;
 }
 
 #include "gt-passes.h"