From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30385 invoked by alias); 16 Oct 2012 18:14:46 -0000 Received: (qmail 30364 invoked by uid 22791); 16 Oct 2012 18:14:45 -0000 X-SWARE-Spam-Status: No, hits=-3.4 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from mx02.qsc.de (HELO mx02.qsc.de) (213.148.130.14) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 16 Oct 2012 18:14:39 +0000 Received: from [192.168.178.22] (port-92-195-26-156.dynamic.qsc.de [92.195.26.156]) by mx02.qsc.de (Postfix) with ESMTP id 35E9B2502D; Tue, 16 Oct 2012 20:14:37 +0200 (CEST) Message-ID: <507DA40C.8030905@net-b.de> Date: Tue, 16 Oct 2012 18:35:00 -0000 From: Tobias Burnus User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:15.0) Gecko/20120825 Thunderbird/15.0 MIME-Version: 1.0 To: Dominique Dhumieres CC: gcc-patches@gcc.gnu.org, fortran@gcc.gnu.org Subject: Re: [Patch, Fortran] Fix OPTIONAL, esp. with polymorphism References: <20121007103940.B9EB93BABA@mailhost.lps.ens.fr> <507D37C2.9070501@net-b.de> <20121016173249.48C953BABA@mailhost.lps.ens.fr> In-Reply-To: <20121016173249.48C953BABA@mailhost.lps.ens.fr> 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: 2012-10/txt/msg01575.txt.bz2 Dominique, Dominique Dhumieres wrote: > The previous class_optional_2.f90 runs after your commit, but it takes > ~168s (so it may have run with the previous patch also, but I was not > patient enough to see it). Here, it takes (current version) < 2s to compile and 0.150s to run the program. If I look at your profile output, the new class_optional_1.f90 should be okay as it doesn't get listed with your profile. The problems seem to occur with elemental calls, which means that probably some additional elemental bugs exist, even if they do not show up in valgrind. Can you confirm that the class_optional_1.f90 of the trunk works correctly? For class_optional_2.f90, we might have to comment some additional elemental calls. I wonder what goes wrong there. (But given that other elemental calls don't work, I do not wonder that much.) Tobias > The culprits are given by the following > profile: > > + 100.0%, start, a.out > | + 100.0%, main, a.out > | | + 100.0%, MAIN__, a.out > | | | + 25.5%, a3ac1.2085, a.out > | | | | 7.2%, s2elem_t.2178, a.out > | | | | 7.0%, s2elem.2184, a.out > | | | + 25.5%, a1a.2151, a.out > | | | | 15.3%, s2elem_t2.2175, a.out > | | | + 24.5%, a1a1.2168, a.out > | | | | 14.2%, s2elem_t.2178, a.out > | | | + 12.2%, a3a.2097, a.out > | | | | 7.1%, s2elem_t2.2175, a.out > | | | + 12.2%, a3a1.2110, a.out > | | | | 7.1%, s2elem_t.2178, a.out >