From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 28029 invoked by alias); 11 Nov 2014 21:43:18 -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 28017 invoked by uid 89); 11 Nov 2014 21:43:18 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 11 Nov 2014 21:43:17 +0000 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id sABLh8x7024471 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 11 Nov 2014 16:43:08 -0500 Received: from [10.3.113.31] (ovpn-113-31.phx2.redhat.com [10.3.113.31]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id sABLh7hx007807; Tue, 11 Nov 2014 16:43:07 -0500 Message-ID: <546282E8.3030703@redhat.com> Date: Tue, 11 Nov 2014 21:43:00 -0000 From: Jeff Law User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Renlin Li , Teresa Johnson CC: Dehao Chen , "gcc-patches@gcc.gnu.org" , Ramana Radhakrishnan , "su@cs.ucdavis.edu" Subject: Re: [PING][PATCH]Partially fix PR61529, bound basic block frequency References: <5450B2C0.6040807@arm.com> <54579F5B.9070700@arm.com> <54594C46.6080704@redhat.com> <545B8F35.9070603@arm.com> <545BB5B5.90008@arm.com> <545BB8F5.8070500@arm.com> <5460EEE7.1090701@arm.com> In-Reply-To: <5460EEE7.1090701@arm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2014-11/txt/msg01092.txt.bz2 On 11/10/14 09:59, Renlin Li wrote: > On 06/11/14 18:07, Renlin Li wrote: >> On 06/11/14 17:59, Teresa Johnson wrote: >>> Thanks for fixing the test case. Can you also add the comment I >>> suggested to the source change? >>> >>>> Please add a comment that this is needed due to insane incoming >>>> frequencies. >>>> >> >> Sorry, I mistakenly add it to the ChangeLog. Should be correct now. >> >> >> x86_64-unknown-linux-gnu bootstrap and regression test have been done, >> no new issue. >> aarch64-none-elf toolchain has been test on the model. No new regression. >> >> gcc/ChangeLog: >> >> 2014-11-06 Renlin Li >> PR middle-end/61529 >> * tree-ssa-threadupdate.c (compute_path_counts): Bound path_in_freq. >> >> gcc/testsuite/ChangeLog: >> >> 2014-11-06 Renlin Li >> PR middle-end/61529 >> * gcc.dg/pr61529.c: New. > > Hi, > > Can anybody help to review or approve this? Per my message on Nov 4, it's approved with the testcase. jeff