From patchwork Wed Jun 30 16:08:54 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: [obvious] Fix fallout from cp/repo.c change
From: Michael Matz <matz@suse.de>
X-Patchwork-Id: 57429
Message-Id: <Pine.LNX.4.64.1006301804590.18619@wotan.suse.de>
To: gcc-patches@gcc.gnu.org
Cc: Nathan Froyd <froydnj@codesourcery.com>
Date: Wed, 30 Jun 2010 18:08:54 +0200 (CEST)

Hi,

I wonder how this could have tested okay, I reliably get segfaults for the 
repo testcases as 'val' will be NULL after the vector got iterated 
through.  I've committed this obvious in r161612.


Ciao,
Michael.


Index: repo.c
===================================================================
--- repo.c      (revision 161611)
+++ repo.c      (working copy)
@@ -269,7 +269,7 @@ finish_repo (void)

   for (ix = VEC_length (tree, pending_repo) - 1;
        VEC_iterate (tree, pending_repo, ix, val);
-       ix--);
+       ix--)
     {
       tree name = DECL_ASSEMBLER_NAME (val);
       char type = IDENTIFIER_REPO_CHOSEN (name) ? 'C' : 'O';