From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24433 invoked by alias); 6 Oct 2008 17:06:37 -0000 Received: (qmail 24423 invoked by uid 22791); 6 Oct 2008 17:06:35 -0000 X-Spam-Check-By: sourceware.org Received: from gv-out-0910.google.com (HELO gv-out-0910.google.com) (216.239.58.187) by sourceware.org (qpsmtpd/0.31) with ESMTP; Mon, 06 Oct 2008 17:05:56 +0000 Received: by gv-out-0910.google.com with SMTP id i36so416886gve.10 for ; Mon, 06 Oct 2008 10:05:52 -0700 (PDT) Received: by 10.86.59.2 with SMTP id h2mr4791401fga.40.1223312752664; Mon, 06 Oct 2008 10:05:52 -0700 (PDT) Received: from ?10.0.0.27? ([92.243.166.74]) by mx.google.com with ESMTPS id l12sm13633726fgb.6.2008.10.06.10.05.49 (version=SSLv3 cipher=RC4-MD5); Mon, 06 Oct 2008 10:05:50 -0700 (PDT) Message-ID: <48EA4569.5010205@ispras.ru> Date: Mon, 06 Oct 2008 17:26:00 -0000 From: Andrey Belevantsev User-Agent: Thunderbird 2.0.0.17 (Windows/20080914) MIME-Version: 1.0 To: sje@cup.hp.com CC: gcc-patches@gcc.gnu.org, wilson@tuliptree.org, vmakarov@redhat.com, Alexander Monakov Subject: Re: Selective scheduling pass - target changes (ia64 & rs6000) [3/3] References: <1222379001.869560.14444.nullmailer@hpsje.cup.hp.com> <48DCDE16.6040702@ispras.ru> <1223071914.10504.20.camel@hpsje.cup.hp.com> In-Reply-To: <1223071914.10504.20.camel@hpsje.cup.hp.com> Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 7bit 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: 2008-10/txt/msg00176.txt.bz2 Steve Ellcey wrote: > I have reviewed the changes on the sel-sched-branch and approve the IA64 > specific changes. I noticed there were a few non-IA64 changes on the branch > and obviously I can't approve those but the IA64 changes look OK. Thanks for the review! I will retest the IA64 changes over the next few days and commit. The non-IA64 changes should be just the difference between scheduling hooks due to the changes we handle speculation. Vlad had already approved those with the main part of sel-sched patch, I just had to revert that pieces when committing the main part without the IA64 part. Of course, if there are other changes, such as bugfixes not yet approved for mainline, I will not commit them. Thanks again, Andrey