From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7037 invoked by alias); 4 Aug 2014 16:39:53 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 7023 invoked by uid 89); 4 Aug 2014 16:39:53 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.5 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD autolearn=ham version=3.3.2 X-HELO: mail3-relais-sop.national.inria.fr Received: from mail3-relais-sop.national.inria.fr (HELO mail3-relais-sop.national.inria.fr) (192.134.164.104) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (CAMELLIA256-SHA encrypted) ESMTPS; Mon, 04 Aug 2014 16:39:38 +0000 Received: from ip-205.net-81-220-140.rev.numericable.fr (HELO laptop-mg.local) ([81.220.140.205]) by mail3-relais-sop.national.inria.fr with ESMTP/TLS/DHE-RSA-AES256-SHA; 04 Aug 2014 18:39:35 +0200 Date: Mon, 04 Aug 2014 16:39:00 -0000 From: Marc Glisse Reply-To: gcc-patches@gcc.gnu.org To: Jakub Jelinek cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Fix bootstrap failure because of -Wreturn-local-addr In-Reply-To: <20140804155509.GN7393@tucnak.redhat.com> Message-ID: References: <20140804155509.GN7393@tucnak.redhat.com> User-Agent: Alpine 2.11 (DEB 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-SW-Source: 2014-08/txt/msg00280.txt.bz2 On Mon, 4 Aug 2014, Jakub Jelinek wrote: > I've tried to bootstrap with r213515 because later revisions broke because > of the hash_map and Ada changes, but unfortunately even that revision > failus to bootstrap, the new -Wreturn-local-addr warning rightfully > warns about get_ivts_expr possibly returning address of a local variable. > > It seems that n_loc is always 1 and loc[0] is always 1, already since > 4.0 when these loop-unroll.c changes have been introduced, so I think the > best fix is just to remove those two fields. Thanks, that will fix PR 62005. I'll try to remember to test with RTL checking for the next warning patch. -- Marc Glisse