From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15519 invoked by alias); 28 Nov 2011 14:18:26 -0000 Received: (qmail 15505 invoked by uid 22791); 28 Nov 2011 14:18:25 -0000 X-SWARE-Spam-Status: No, hits=-2.9 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00 X-Spam-Check-By: sourceware.org Received: from localhost (HELO gcc.gnu.org) (127.0.0.1) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 28 Nov 2011 14:18:10 +0000 From: "burnus at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug fortran/51308] PARAMETER attribute conflicts with SAVE attribute Date: Mon, 28 Nov 2011 14:24:00 -0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: fortran X-Bugzilla-Keywords: rejects-valid X-Bugzilla-Severity: normal X-Bugzilla-Who: burnus at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated Content-Type: text/plain; charset="UTF-8" MIME-Version: 1.0 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org X-SW-Source: 2011-11/txt/msg02652.txt.bz2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51308 --- Comment #5 from Tobias Burnus 2011-11-28 14:17:50 UTC --- (In reply to comment #4) > I kept the SAVE attribute because the comment (which you remove > in your patch) claims that it is needed to actually allow the > the compiler to initialize c_null_ptr and c_null_funptr to NULL. Well, an initialization (sym->value) can only be only used for PARAMETERs or for variables in static memory ("SAVE"). Having both "SAVE" and "PARAMETER" does not really make sense. I assume that first the SAVE was added - and only later PARAMETER (with a similar comment and many lines later in the file). I have submitted the patch for approval at http://gcc.gnu.org/ml/fortran/2011-11/msg00234.html and given your approval in comment 4 I will commit it. Thanks for looking through my patch and for debugging and creating the draft patch of comment 2.