From patchwork Mon Aug 23 06:36:14 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kai Tietz X-Patchwork-Id: 62448 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id BAE88B70CB for ; Mon, 23 Aug 2010 16:36:23 +1000 (EST) Received: (qmail 5530 invoked by alias); 23 Aug 2010 06:36:22 -0000 Received: (qmail 5520 invoked by uid 22791); 23 Aug 2010 06:36:21 -0000 X-SWARE-Spam-Status: No, hits=-0.9 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM, RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from mail-qy0-f182.google.com (HELO mail-qy0-f182.google.com) (209.85.216.182) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 23 Aug 2010 06:36:16 +0000 Received: by qyk4 with SMTP id 4so5715100qyk.20 for ; Sun, 22 Aug 2010 23:36:14 -0700 (PDT) MIME-Version: 1.0 Received: by 10.229.89.15 with SMTP id c15mr1344926qcm.202.1282545374163; Sun, 22 Aug 2010 23:36:14 -0700 (PDT) Received: by 10.229.45.13 with HTTP; Sun, 22 Aug 2010 23:36:14 -0700 (PDT) Date: Mon, 23 Aug 2010 08:36:14 +0200 Message-ID: Subject: [patch]: Possibly uninitialized variable in tree-vect-loop.c From: Kai Tietz To: GCC Patches X-IsSubscribed: yes Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Hi, Somehow the diagnostic of 4.6 about possible unitialized variables is a bit vague. By using an older gcc for bootstrapping I found a case in tree-vect-loop.c, which indeed has unitialized variable in some branches. This patch fixes it. ChangeLog 2010-08-23 Kai Tietz * tree-vect-loop.c (vect_get_single_scalar_iteraion_cost): Pre-initialize innerloop_iters to one. Tested for x86_64-pc-linux-gnu, x86_64-w64-mingw32, and i686-w64-mingw32. Ok for apply? Regards, Kai Index: tree-vect-loop.c =================================================================== --- tree-vect-loop.c (revision 163445) +++ tree-vect-loop.c (working copy) @@ -2031,6 +2031,7 @@ statements. */ /* FORNOW. */ + innerloop_iters = 1; if (loop->inner) innerloop_iters = 50; /* FIXME */