From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 42734 invoked by alias); 29 May 2017 19:33:35 -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 42709 invoked by uid 89); 29 May 2017 19:33:34 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,RCVD_IN_SORBS_SPAM,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy=H*i:sk:2c00d8e, H*f:sk:2c00d8e, H*MI:sk:2c00d8e X-Spam-User: qpsmtpd, 2 recipients X-HELO: mtaout002-public.msg.strl.va.charter.net Received: from mtaout002-public.msg.strl.va.charter.net (HELO mtaout002-public.msg.strl.va.charter.net) (68.114.190.27) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 29 May 2017 19:33:33 +0000 Received: from impout003 ([68.114.189.18]) by mtaout002.msg.strl.va.charter.net (InterMail vM.9.00.023.01 201-2473-194) with ESMTP id <20170529193335.QVWO7360.mtaout002.msg.strl.va.charter.net@impout003>; Mon, 29 May 2017 14:33:35 -0500 Received: from amda8.localdomain ([76.178.132.41]) by impout003 with charter.net id SKZa1v0030tkpWs01KZaMd; Mon, 29 May 2017 14:33:35 -0500 X-Authority-Analysis: v=2.2 cv=XZf59Mx5 c=1 sm=1 tr=0 a=Y8XvaKYZbF3VPrlWiqOmuQ==:117 a=Y8XvaKYZbF3VPrlWiqOmuQ==:17 a=IkcTkHD0fZMA:10 a=AcUdcMT6eyoTGUhDVtEA:9 a=QEXdDO2ut3YA:10 X-Auth-id: anZkZWxpc2xlQGNoYXJ0ZXIubmV0 Subject: Re: [patch, libgfortran] PR53029 missed optimization in internal read (without implied-do-loop) To: Thomas Koenig , "fortran@gcc.gnu.org" Cc: GCC Patches References: <2c00d8e7-700e-c13e-dbb2-d1928cc3c482@netcologne.de> From: Jerry DeLisle Message-ID: <4e6bea81-fa89-da95-6be4-56549e06f695@charter.net> Date: Mon, 29 May 2017 19:44:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 MIME-Version: 1.0 In-Reply-To: <2c00d8e7-700e-c13e-dbb2-d1928cc3c482@netcologne.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2017-05/txt/msg02212.txt.bz2 On 05/29/2017 09:51 AM, Thomas Koenig wrote: > Hi Jerry, > >> Regression tested on x86_64. I have added a test case which will check the >> execution time of the loop. The previous results of the REAd were correct, >> just took a long time on large arrays. >> >> OK for trunk? > > OK. > > It might be good if you followed Manfred's suggestion and turned > down the timeout to something like 0.5 seconds. > > Thanks for the patch! > > I would also consider backporting, the speedup is just so > large. What do others think? > > Regards > > Thomas Committed. A gcc/testsuite/gfortran.dg/read_5.f90 M gcc/testsuite/ChangeLog M libgfortran/ChangeLog M libgfortran/io/list_read.c Committed r248577 Thanks, Jerry