From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-patches-return-408787-listarch-gcc-patches=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 110146 invoked by alias); 1 Oct 2015 06:42:26 -0000
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-patches.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-patches/>
List-Post: <mailto:gcc-patches@gcc.gnu.org>
List-Help: <mailto:gcc-patches-help@gcc.gnu.org>
Sender: gcc-patches-owner@gcc.gnu.org
Received: (qmail 110135 invoked by uid 89); 1 Oct 2015 06:42:25 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-0.9 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,T_RP_MATCHES_RCVD autolearn=no version=3.3.2
X-HELO: mail2-relais-roc.national.inria.fr
Received: from mail2-relais-roc.national.inria.fr (HELO mail2-relais-roc.national.inria.fr) (192.134.164.83) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Thu, 01 Oct 2015 06:42:23 +0000
Received: from 89-158-40-219.rev.numericable.fr (HELO laptop-mg.local) ([89.158.40.219])  by mail2-relais-roc.national.inria.fr with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Oct 2015 08:42:06 +0200
Date: Thu, 01 Oct 2015 06:42:00 -0000
From: Marc Glisse <marc.glisse@inria.fr>
Reply-To: gcc-patches@gcc.gnu.org
To: Michael Collison <michael.collison@linaro.org>
cc: gcc-patches@gcc.gnu.org, Richard Biener <richard.guenther@gmail.com>,     Jeff Law <law@redhat.com>
Subject: Re: [PATCH] Optimize certain end of loop conditions into min/max operation
In-Reply-To: <560CC744.4040704@linaro.org>
Message-ID: <alpine.DEB.2.20.1510010838460.1725@laptop-mg.saclay.inria.fr>
References: <55B5A884.4060105@linaro.org> <CAFiYyc1ObfTFaChzLiAezxEzVziVFnDzCTBhvS0v975-xXRosw@mail.gmail.com> <55B65A4B.3050705@redhat.com> <CAFiYyc2J8zo6cVVSj-06+DHoELacugJTeTz1ezqgva2cD44hYA@mail.gmail.com> <55BBA052.2060900@redhat.com> <55BBBBE6.2070207@linaro.org> <55BBBE2E.1020408@redhat.com> <55FBAFD1.9080300@linaro.org> <alpine.DEB.2.20.1509180850360.1946@laptop-mg.saclay.inria.fr> <alpine.DEB.2.20.1509180931340.1946@laptop-mg.saclay.inria.fr> <CAFiYyc3Y1LJqQDfK12BuHNLU=QAb1fjZCfNRmG9L_fpvfPtt6g@mail.gmail.com> <560B8F51.6030108@linaro.org> <CAFiYyc2XfPQk7gRB9QTX2r=R2FaL5O4+YNqQTXEyQnudsBzrbA@mail.gmail.com> <560C0D65.2010300@linaro.org> <alpine.DEB.2.20.1509302122240.1747@laptop-mg.saclay.inria.fr> <560CC744.4040704@linaro.org>
User-Agent: Alpine 2.20 (DEB 67 2015-01-07)
MIME-Version: 1.0
Content-Type: text/plain; format=flowed; charset=US-ASCII
X-SW-Source: 2015-10/txt/msg00008.txt.bz2

On Wed, 30 Sep 2015, Michael Collison wrote:

> Richard and Marc,
>
> Latest patch attached which incorporates all comments.
>
> 2015-09-30  Michael Collison <michael.collison@linaro.org>
>        Andrew Pinski <andrew.pinski@caviumnetworks.com>
>
>    * match.pd ((x < y) && (x < z) -> x < min (y,z),
>    (x > y) and (x > z) -> x > max (y,z))
>    * testsuite/gcc.dg/tree-ssa/minmax-loopend.c: New test.

You are still missing at least the indentation.

+/* { dg-final { scan-tree-dump "MIN_EXPR" 1 "optimized" } } */

I believe it is only scan-tree-dump-times that takes a number. 
scan-tree-dump seems to have only 2 arguments.

-- 
Marc Glisse