From patchwork Sat Feb 22 16:59:21 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hans-Peter Nilsson X-Patchwork-Id: 323204 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 6964D2C022C for ; Sun, 23 Feb 2014 03:59:35 +1100 (EST) DomainKey-Signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :message-id:from:to:subject:mime-version:content-type :content-transfer-encoding; q=dns; s=default; b=cAZXd6v26K1YuK5a owwU0QIOD+u39EZaH3NnqmNLcjsy9XDB00qmLnB189DSelYE5xMR8+bbqklnD+y9 GBkqjuOnRDUlq75KCU9qaCrPC1Y+eXTgwHrvahABcYwpxu29Bq3a9vX0rNlrQGpM o3t2oPR34fgY+IM0XBmJVAl3+70= 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 :message-id:from:to:subject:mime-version:content-type :content-transfer-encoding; s=default; bh=yMg/Y7n7dMJ3QsGKOjm/xl 4X9jg=; b=xFTdW6a187+WL9EkZWajn7HLfC1mge1geSi83wtqN5bvCwFc7lVuvO OS6TxSp1lTvVlbmZp3p1PJE3+L8ovNCDPP1FMUjRwEtJWfY0quf5ZGfG5ZYTRVr1 6tJECcjD2hIy9A+qzEBMQoqqjAFC49EKJclINcXhy9tTSPLON6fUI= Received: (qmail 1631 invoked by alias); 22 Feb 2014 16:59:28 -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 1621 invoked by uid 89); 22 Feb 2014 16:59:27 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=AWL, BAYES_00, RP_MATCHES_RCVD, SPF_PASS autolearn=ham version=3.3.2 X-HELO: bastet.se.axis.com Received: from bastet.se.axis.com (HELO bastet.se.axis.com) (195.60.68.11) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sat, 22 Feb 2014 16:59:26 +0000 Received: from localhost (localhost [127.0.0.1]) by bastet.se.axis.com (Postfix) with ESMTP id 081D51824D for ; Sat, 22 Feb 2014 17:59:23 +0100 (CET) Received: from bastet.se.axis.com ([IPv6:::ffff:127.0.0.1]) by localhost (bastet.se.axis.com [::ffff:127.0.0.1]) (amavisd-new, port 10024) with LMTP id v1qry1xUHQpe for ; Sat, 22 Feb 2014 17:59:22 +0100 (CET) Received: from boulder.se.axis.com (boulder.se.axis.com [10.0.2.104]) by bastet.se.axis.com (Postfix) with ESMTP id 75C2C18249 for ; Sat, 22 Feb 2014 17:59:22 +0100 (CET) Received: from boulder.se.axis.com (localhost [127.0.0.1]) by postfix.imss71 (Postfix) with ESMTP id 5E0FEC5F for ; Sat, 22 Feb 2014 17:59:22 +0100 (CET) Received: from seth.se.axis.com (seth.se.axis.com [10.0.2.172]) by boulder.se.axis.com (Postfix) with ESMTP id 526149DD for ; Sat, 22 Feb 2014 17:59:22 +0100 (CET) Received: from ignucius.se.axis.com (ignucius.se.axis.com [10.88.21.50]) by seth.se.axis.com (Postfix) with ESMTP id 47DC83E048; Sat, 22 Feb 2014 17:59:22 +0100 (CET) Received: from ignucius.se.axis.com (localhost [127.0.0.1]) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) with ESMTP id s1MGxLh9010563; Sat, 22 Feb 2014 17:59:22 +0100 Received: (from hp@localhost) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) id s1MGxLvw010559; Sat, 22 Feb 2014 17:59:21 +0100 Date: Sat, 22 Feb 2014 17:59:21 +0100 Message-Id: <201402221659.s1MGxLvw010559@ignucius.se.axis.com> From: Hans-Peter Nilsson To: gcc-patches@gcc.gnu.org Subject: Committed: Add CRIS to logical_op_short_circuit MIME-Version: 1.0 There was a new effective-target predicate (thanks, Richard S), but the droplet that broke the camel's back or something, wasn't added to its target-list. Committed after brief testing (checking that tests fail before, checking that tests pass after patch). Other observations: - LOGICAL_OP_NON_SHORT_CIRCUIT should move into defaults.h instead of the identical copies in fold-const.c and tree-ssa-ifcombine.c (both default to BRANCH_COST >= 2). - There seem to be more targets to add to that list (requires grep and test) and more tests that should use the predicate (requires at least brief analysis). gcc/testsuite: PR testsuite/60173 * lib/target-supports.exp (check_effective_target_logical_op_short_circuit): Add cris-*-* and crisv32-*-* to list. brgds, H-P Index: gcc/testsuite/lib/target-supports.exp =================================================================== --- gcc/testsuite/lib/target-supports.exp (revision 208041) +++ gcc/testsuite/lib/target-supports.exp (working copy) @@ -5696,6 +5696,7 @@ proc check_effective_target_logical_op_s if { [istarget mips*-*-*] || [istarget arc*-*-*] || [istarget avr*-*-*] + || [istarget crisv32-*-*] || [istarget cris-*-*] || [check_effective_target_arm_cortex_m] } { return 1 }