From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18145 invoked by alias); 27 Sep 2011 12:57:09 -0000 Received: (qmail 18136 invoked by uid 22791); 27 Sep 2011 12:57:08 -0000 X-SWARE-Spam-Status: No, hits=-1.5 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_BL_SPAMCOP_NET,RCVD_IN_DNSWL_LOW,TW_CF,TW_FC X-Spam-Check-By: sourceware.org Received: from mail-gy0-f175.google.com (HELO mail-gy0-f175.google.com) (209.85.160.175) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 27 Sep 2011 12:56:50 +0000 Received: by gyg8 with SMTP id 8so5768516gyg.20 for ; Tue, 27 Sep 2011 05:56:50 -0700 (PDT) MIME-Version: 1.0 Received: by 10.151.5.15 with SMTP id h15mr2408302ybi.295.1317128210129; Tue, 27 Sep 2011 05:56:50 -0700 (PDT) Received: by 10.150.177.4 with HTTP; Tue, 27 Sep 2011 05:56:49 -0700 (PDT) In-Reply-To: <4E80FCEE.2030304@redhat.com> References: <4E4ADDAC.4090006@redhat.com> <4E4D8B54.1050409@redhat.com> <4E80FCEE.2030304@redhat.com> Date: Tue, 27 Sep 2011 13:51:00 -0000 Message-ID: Subject: Re: RFA: Avoiding unprofitable speculation From: Richard Guenther To: Jeff Law Cc: Richard Henderson , gcc-patches Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes 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-09/txt/msg01723.txt.bz2 On Tue, Sep 27, 2011 at 12:30 AM, Jeff Law wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 08/18/11 15:59, Richard Henderson wrote: >> On 08/17/2011 12:21 AM, Richard Guenther wrote: >>> The patch itself looks sensible, though I am surprised ifcvt >>> doesn't run in cfglayout mode (so you have to use reg notes to >>> find probabilities ...) >> >> It does run in cfglayout mode. >> >> Jeff, I believe you're supposed to get the probabilities from some >> combination of >> >> bb->frequency edge->probability EDGE_FREQUENCY(edge) > OK. =A0Here's the revised patch. =A0There's other places in ifcvt.c that > utilize the notes that I didn't modify. > > Bootstrapped & regression tested x86_64-unknown-linux-gnu. =A0Also > verified performance data hasn't changed materially. > > OK for trunk? Looks good to me. Thanks, Richard. > Thanks, > jeff > > > -----BEGIN PGP SIGNATURE----- > Version: GnuPG v1.4.11 (GNU/Linux) > Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/ > > iQEcBAEBAgAGBQJOgPzuAAoJEBRtltQi2kC7uQkH+wTyLl3MwKkqC11ozyOAVSQ9 > CrlwLAuN7JV7kZcguKZ9GpXYHePZHZHCISXaVv3LQQBXnE7PehJOWsd1D5BQRv2/ > eqVHIAOYg0LamY2cRiWW8pKMiMjs7vb9q0fiehQGg0zxAJMc9crBjwLPGFjZAksw > UNzzon/NKfSMYsz9X/olDfk8DPa1DmAjBnNOcHzKLGdx7KDa6Npo20k3D/PwDbIe > y1Ff9pZBXJP6tNU+0cn9lyyt+w6ghFQRkpKJoJ6iSOxKQ6v23+03o4sT5GHv3Gvy > 07R6NJU9vqt7a9GvxcyJ9BsOlCCJ/pA/4lProHrdcAZrOYWZAA4uoJeDBPOEq+g=3D > =3Da5LI > -----END PGP SIGNATURE----- >