From patchwork Sat Sep 24 05:46:15 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Pinski X-Patchwork-Id: 674288 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 3sgzkJ3slmz9s3s for ; Sat, 24 Sep 2016 15:46:29 +1000 (AEST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender :mime-version:from:date:message-id:subject:to:content-type; q= dns; s=default; b=v1Hhe/zhP2qs5ozCuLveHNeHsG6rhnt+WZzO9lieoQRmbH bVpdm/kpHnliO3BrO1fHHZrHdVIcCFZbjzEs9MDwm23SjD1QrhTzA7TSED5Gjh3S crLupBTAMczk/8aiIQhU92mfYs9vhECwdWSuF9z5DPWSw3D4SCd7pVWseDFOs= 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 :mime-version:from:date:message-id:subject:to:content-type; s= default; bh=3wFSL6wAWnZ9io2U9it7xL+AoaM=; b=f8RevpnljPP5y+jmgab8 P0tSJa1gRY1ehAUl7UjSDHLlYr+bXj5kn02tnjmzz2bNIwWvFjI20Az8zbsQzMQ0 fOio3jpy44u9GGTVXzHzjttAOtDORGMEEUUlg4zU6AKpUpcTY71gjlro0bxBbDvi 4setkWNlzsWwBqsZYjXNba4= Received: (qmail 92209 invoked by alias); 24 Sep 2016 05:46:22 -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 92189 invoked by uid 89); 24 Sep 2016 05:46:21 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.3 required=5.0 tests=AWL, BAYES_00, FREEMAIL_FROM, KAM_ASCII_DIVIDERS, RCVD_IN_DNSWL_LOW, RCVD_IN_SORBS_SPAM, SPF_PASS autolearn=no version=3.3.2 spammy=HX-Envelope-From:sk:pinskia, reduces, 1677, back-end X-HELO: mail-lf0-f49.google.com Received: from mail-lf0-f49.google.com (HELO mail-lf0-f49.google.com) (209.85.215.49) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 24 Sep 2016 05:46:18 +0000 Received: by mail-lf0-f49.google.com with SMTP id b71so80295720lfg.0 for ; Fri, 23 Sep 2016 22:46:18 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:from:date:message-id:subject:to; bh=Wck5cpNcxWe44wgXhzINFdkfGrpLIs6BYw6ZozCN+ys=; b=QhhlLuYwiUwpDHTalcHbBYaUV5p9l0YhKDZxJJYAKlaQgOr+F/md3rZOeRHBfnIX+L BL/mzUvJiR1Gmp+TwsrqyTjV31CEF2yYgZWVmb4rKUXS5qz7+l2e6flbRNzohrD0dCkn b+LSYQPyn3rjXOGXiRybb8CMBsCPoTlPasrT9ImPLEP28Yeb/i3/jIIwuoVx6OBeI4QL Xh89SIiFsIqCYQN5YGnB8xf46bz/rsdZPaVPAWlM5jM43rSkH08sYraCfB5Mvk8QirCL ckVcIANV4UwsX31RkcPkOyVaqUTftOok/jcuSXcYK3WAC3FmC8eSwfsSJc30njn3gUtW t6Qg== X-Gm-Message-State: AE9vXwO21FKc2stnIx05oVcqyTu2eEv9QkfiHYLjqHbx2MmKwdXt/5sNhM+Fr6v7ugmRUcmKfP/054pDh9lwMA== X-Received: by 10.25.21.162 with SMTP id 34mr4661878lfv.100.1474695976202; Fri, 23 Sep 2016 22:46:16 -0700 (PDT) MIME-Version: 1.0 Received: by 10.25.166.76 with HTTP; Fri, 23 Sep 2016 22:46:15 -0700 (PDT) From: Andrew Pinski Date: Sat, 24 Sep 2016 13:46:15 +0800 Message-ID: Subject: [PATCH/AARCH64] Have the verbose cost model output output be controllable To: GCC Patches X-IsSubscribed: yes Hi, As reported in PR 61367, the aarch64 back-end is too verbose when it is dealing with the cost model. I tend to agree, no other back-end is this verbose. So I decided to add an option to enable this verbose output if requested. I did NOT document it in invoke.texi because I don't feel like this is an option which an user should use. But I can add it if requested. OK? Bootstrapped and tested on aarch64-linux-gnu with no regressions. Thanks, Andrew Pinski ChangeLog: * config/aarch64/aarch64.opt (mverbose-cost-dump): New option. * config/aarch64/aarch64.c (aarch64_rtx_costs): Use flag_aarch64_verbose_cost instead of checking for details dump. (aarch64_rtx_costs_wrapper): Likewise. Index: config/aarch64/aarch64.c =================================================================== --- config/aarch64/aarch64.c (revision 240461) +++ config/aarch64/aarch64.c (working copy) @@ -7351,7 +7351,8 @@ break; } - if (dump_file && (dump_flags & TDF_DETAILS)) + if (dump_file + && flag_aarch64_verbose_cost) fprintf (dump_file, "\nFailed to cost RTX. Assuming default cost.\n"); @@ -7367,7 +7368,8 @@ { bool result = aarch64_rtx_costs (x, mode, outer, param, cost, speed); - if (dump_file && (dump_flags & TDF_DETAILS)) + if (dump_file + && flag_aarch64_verbose_cost) { print_rtl_single (dump_file, x); fprintf (dump_file, "\n%s cost: %d (%s)\n", Index: config/aarch64/aarch64.opt =================================================================== --- config/aarch64/aarch64.opt (revision 240461) +++ config/aarch64/aarch64.opt (working copy) @@ -167,3 +167,7 @@ Enable the division approximation. Enabling this reduces precision of division results to about 16 bits for single precision and to 32 bits for double precision. + +mverbose-cost-dump +Common Var(flag_aarch64_verbose_cost) +Enables verbose cost model dummping in the debug dump files.