Hi!

I'd like to apply the following patch to fix a warning:

../../../../gcc/gcc/varasm.c:2094:13: warning: 
‘pending_assemble_externals_processed’ defined but not used [-Wunused-variable]

(seen on microblaze-uclinux and pdp11-aout) introduced with a fix for
PR middle-end/55709 and 52894:

+2012-12-16  John David Anglin  <dave.ang...@nrc-cnrc.gc.ca>
+
+       PR middle-end/55709
+       Forward port from 4.7 branch:
+       2012-04-10  John David Anglin  <dave.ang...@nrc-cnrc.gc.ca>
+
+       PR middle-end/52894
+       * varasm.c (process_pending_assemble_externals): Set
+       pending_assemble_externals_processed true.
+       (assemble_external): Call assemble_external_real if the pending
+       assemble externals have been processed.

Okay to commit?



2012-12-17  Jan-Benedict Glaw  <jbg...@lug-owl.de>

        * gcc/varasm.c (pending_assemble_externals_processed): Guard
        with ASM_OUTPUT_EXTERNAL.

diff --git a/gcc/varasm.c b/gcc/varasm.c
index daeb6e2..53ebfbf 100644
--- a/gcc/varasm.c
+++ b/gcc/varasm.c
@@ -2088,12 +2088,12 @@ contains_pointers_p (tree type)
    it all the way to final.  See PR 17982 for further discussion.  */
 static GTY(()) tree pending_assemble_externals;
 
+#ifdef ASM_OUTPUT_EXTERNAL
 /* Some targets delay some output to final using TARGET_ASM_FILE_END.
    As a result, assemble_external can be called after the list of externals
    is processed and the pointer set destroyed.  */
 static bool pending_assemble_externals_processed;
 
-#ifdef ASM_OUTPUT_EXTERNAL
 /* Avoid O(external_decls**2) lookups in the pending_assemble_externals
    TREE_LIST in assemble_external.  */
 static struct pointer_set_t *pending_assemble_externals_set;



-- 
      Jan-Benedict Glaw      jbg...@lug-owl.de              +49-172-7608481
Signature of:          GDB has a 'break' feature; why doesn't it have 'fix' too?
the second  :

Attachment: signature.asc
Description: Digital signature

Reply via email to