From patchwork Tue May 31 10:54:40 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jan Hubicka X-Patchwork-Id: 628132 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 3rJr3h6FN7z9s3s for ; Tue, 31 May 2016 20:54:52 +1000 (AEST) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=gcc.gnu.org header.i=@gcc.gnu.org header.b=Cx1MR6Ur; dkim-atps=neutral DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:subject:message-id:mime-version:content-type; q=dns; s= default; b=iR1XNsnMMRb1lFJyoVtvcbYZtUpxo0WPiKj37iQxp+6BqDOoFqIs/ kQhyqyCy4mO4MQV0IMZ5qsk2OOzci5hN+H6tZ/QQTiGhJOo+FlA0qlAzdsc6ksD/ WZQBLgNyNVm7Wxv+bWGrg8eTI90C8MZ2wn+Vj5hmRrR9jAApOhK3SQ= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:subject:message-id:mime-version:content-type; s= default; bh=R5/w9vDWMbstQAjGJQDxvVL3YVo=; b=Cx1MR6UrdxcrIXIfpir/ vNz2LL0WQRLEeItci0o1hroCXE8ROWE0j0vkQTawjRgRM7PXfnzn0Bt6Cg+s42m5 Gav/Vm+QsxEprovJkyCW/jpLk4SnERtD8aMIYDoSkwKeO3jWXunB1OSF0s1SiOk6 A6myQ9CUKMtYsU/ccS4zYIc= Received: (qmail 34706 invoked by alias); 31 May 2016 10:54:45 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Received: (qmail 34695 invoked by uid 89); 31 May 2016 10:54:44 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.2 required=5.0 tests=AWL, BAYES_20, KAM_ASCII_DIVIDERS, KAM_LAZY_DOMAIN_SECURITY, RP_MATCHES_RCVD autolearn=no version=3.3.2 spammy=@opindex, flag_peel_loops, Enabled, imply X-HELO: nikam.ms.mff.cuni.cz Received: from nikam.ms.mff.cuni.cz (HELO nikam.ms.mff.cuni.cz) (195.113.20.16) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Tue, 31 May 2016 10:54:43 +0000 Received: by nikam.ms.mff.cuni.cz (Postfix, from userid 16202) id 65B525453C0; Tue, 31 May 2016 12:54:40 +0200 (CEST) Date: Tue, 31 May 2016 12:54:40 +0200 From: Jan Hubicka To: gcc-patches@gcc.gnu.org, rguenther@suse.de Subject: Do not imply -frename-registers by -fpeel-loops Message-ID: <20160531105438.GB95630@kam.mff.cuni.cz> MIME-Version: 1.0 Content-Disposition: inline User-Agent: Mutt/1.5.21 (2010-09-15) Hi, this patch makes -frename-registers independent with -fpeel-loops. I think the original idea for building this in was the fact that loop peeling creates large basic blocks where scheduling matters. This is no longer true about -fpeel-loops because this happens only for complete peeling which has been moved to separate pass long time ago. Bootsrapping/regtesting x86_64-linux, OK? * doc/invoke.texi (-frename-registers): Drop -fpeel-loops from list of flags impliying the register renaming. * toplev.c (process_options): Do not imply flag_rename_registers with loop peeling. Index: doc/invoke.texi =================================================================== --- doc/invoke.texi (revision 236914) +++ doc/invoke.texi (working copy) @@ -8623,7 +8623,7 @@ debug information format adopted by the make debugging impossible, since variables no longer stay in a ``home register''. -Enabled by default with @option{-funroll-loops} and @option{-fpeel-loops}. +Enabled by default with @option{-funroll-loops}. @item -fschedule-fusion @opindex fschedule-fusion Index: toplev.c =================================================================== --- toplev.c (revision 236915) +++ toplev.c (working copy) @@ -1299,7 +1299,7 @@ process_options (void) flag_web = flag_unroll_loops; if (flag_rename_registers == AUTODETECT_VALUE) - flag_rename_registers = flag_unroll_loops || flag_peel_loops; + flag_rename_registers = flag_unroll_loops; if (flag_non_call_exceptions) flag_asynchronous_unwind_tables = 1;