This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: match.pd patch: min(-x, -y), min(~x, ~y)

From: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
To: Marc Glisse <marc dot glisse at inria dot fr>
Cc: gcc-patches at gcc dot gnu dot org
Date: Fri, 22 Apr 2016 10:43:09 +0100
Subject: Re: match.pd patch: min(-x, -y), min(~x, ~y)
Authentication-results: sourceware.org; auth=none
References: <alpine dot DEB dot 2 dot 02 dot 1604211217410 dot 22599 at laptop-mg dot saclay dot inria dot fr> <5719EC75 dot 2040509 at foss dot arm dot com> <alpine dot DEB dot 2 dot 20 dot 1604221139160 dot 1900 at laptop-mg dot saclay dot inria dot fr>


On 22/04/16 10:42, Marc Glisse wrote:

On Fri, 22 Apr 2016, Kyrill Tkachov wrote:

2016-04-21  Marc Glisse <marc.glisse@inria.fr>

gcc/
    * match.pd (min(-x, -y), max(-x, -y), min(~x, ~y), max(~x, ~y)):
    New transformations.

gcc/testsuite/
    * gcc.dg/tree-ssa/minmax-2.c: New testcase.


I see the new testcase failing on aarch64:
FAIL: gcc.dg/tree-ssa/minmax-2.c scan-tree-dump optimized "__builtin_fmin"


Strange, it seems to work in https://gcc.gnu.org/ml/gcc-testresults/2016-04/msg02120.html

Is that on some freestanding kind of setup where the builtin might be disabled?


Ah, this is aarch64-none-elf which uses newlib as the C library.
Let me check on aarch64-none-linux-gnu and get back to you.

Thanks,
Kyrill

Follow-Ups:
- Re: match.pd patch: min(-x, -y), min(~x, ~y)
  - From: Kyrill Tkachov

References:
- match.pd patch: min(-x, -y), min(~x, ~y)
  - From: Marc Glisse
- Re: match.pd patch: min(-x, -y), min(~x, ~y)
  - From: Kyrill Tkachov
- Re: match.pd patch: min(-x, -y), min(~x, ~y)
  - From: Marc Glisse

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]