From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17845 invoked by alias); 10 Jun 2011 17:37:33 -0000 Received: (qmail 17831 invoked by uid 22791); 10 Jun 2011 17:37:32 -0000 X-SWARE-Spam-Status: No, hits=-1.6 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,RFC_ABUSE_POST,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mta21.charter.net (HELO mta21.charter.net) (216.33.127.81) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 10 Jun 2011 17:37:19 +0000 Received: from imp09 ([10.20.200.9]) by mta21.charter.net (InterMail vM.7.09.02.04 201-2219-117-106-20090629) with ESMTP id <20110610173718.HWHZ11595.mta21.charter.net@imp09>; Fri, 10 Jun 2011 13:37:18 -0400 Received: from quava.localdomain ([68.186.96.128]) by imp09 with smtp.charter.net id uHdG1g00E2mBQ6U05HdHAW; Fri, 10 Jun 2011 13:37:18 -0400 X-Authority-Analysis: v=1.1 cv=1b2X7W/SifksZeClH/haT1SUt4udqxFGF00pZw2/jJk= c=1 sm=1 a=qd9eegJWHRQA:10 a=JtzazXA-K3gA:10 a=yUnIBFQkZM0A:10 a=8nJEP1OIZ-IA:10 a=W3wzxSix39fL1hz8i2isiQ==:17 a=9dG_XchR0bPH7FQi220A:9 a=wPNLvfGTeEIA:10 a=W3wzxSix39fL1hz8i2isiQ==:117 Message-ID: <4DF2565E.9020801@charter.net> Date: Fri, 10 Jun 2011 17:45:00 -0000 From: jerry DeLisle User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc15 Thunderbird/3.1.10 MIME-Version: 1.0 To: Tobias Burnus CC: gcc patches , gfortran Subject: Re: [Patch, Fortran] PR 49324: Deep copy array constr of DT with allocatable components References: <4DF20A1D.2000709@net-b.de> In-Reply-To: <4DF20A1D.2000709@net-b.de> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit 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 X-SW-Source: 2011-06/txt/msg00865.txt.bz2 On 06/10/2011 05:12 AM, Tobias Burnus wrote: > This patch fixes parts of the PR 49324: There was a deep copy missing for > assigning an array constructor of a DT with allocatable components. > > Whether a deep copy is done, depends on a flag. I think the flag has been added > to avoid a deep copy and multiple evaluation for functions, which return DT w/ > allocatable components, and for user-defined operators. > > Remains to do be done: > - RESHAPE is mishandled (design error): No deep copy. > - Reallocate on assignment fails (missing NULL initialization of the malloced > memory). > > Build and regtested on x86-64-linux. OK? > > Tobias OK, thanks for patch. Jerry