From patchwork Thu Dec 22 14:25:19 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Uros Bizjak X-Patchwork-Id: 132854 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]) by ozlabs.org (Postfix) with SMTP id DF16DB717C for ; Fri, 23 Dec 2011 01:25:58 +1100 (EST) Received: (qmail 12243 invoked by alias); 22 Dec 2011 14:25:55 -0000 Received: (qmail 12221 invoked by uid 22791); 22 Dec 2011 14:25:54 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL, BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, FREEMAIL_FROM, RCVD_IN_DNSWL_LOW, TW_ZJ 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; Thu, 22 Dec 2011 14:25:20 +0000 Received: by ghrr17 with SMTP id r17so3801523ghr.20 for ; Thu, 22 Dec 2011 06:25:19 -0800 (PST) MIME-Version: 1.0 Received: by 10.101.131.6 with SMTP id i6mr4655005ann.21.1324563919352; Thu, 22 Dec 2011 06:25:19 -0800 (PST) Received: by 10.147.133.6 with HTTP; Thu, 22 Dec 2011 06:25:19 -0800 (PST) In-Reply-To: References: Date: Thu, 22 Dec 2011 15:25:19 +0100 Message-ID: Subject: [PATCH v2, alpha]: Enable flag_ree, fix PR 27468 From: Uros Bizjak To: "Joseph S. Myers" Cc: gcc-patches@gcc.gnu.org, Richard Henderson , Ilya Enkovich 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 On Thu, Dec 22, 2011 at 1:58 PM, Joseph S. Myers wrote: > Is there a reason you are doing it this way rather than by adding an entry > to alpha_option_optimization_table (in > common/config/alpha/alpha-common.c), which would generally be the > preferred way of enabling an option at certain optimization levels on a > target-specific basis? This patch updates flag setting approach according to Joseph's comments. 2011-12-22 Uros Bizjak PR target/27468 * common/config/alpha/alpha-common.c (alpha_option_optimization_table): Enable flag_ree at -O2 or higher. Patch was bootstrapped and regression tested on alphaev68-pc-linux-gnu. OK for mainline? Uros. Index: common/config/alpha/alpha-common.c =================================================================== --- common/config/alpha/alpha-common.c (revision 182617) +++ common/config/alpha/alpha-common.c (working copy) @@ -33,6 +33,8 @@ static const struct default_options alpha_option_optimization_table[] = { { OPT_LEVELS_1_PLUS, OPT_fomit_frame_pointer, NULL, 1 }, + /* Enable redundant extension instructions removal at -O2 and higher. */ + { OPT_LEVELS_2_PLUS, OPT_free, NULL, 1 }, { OPT_LEVELS_NONE, 0, NULL, 0 } };