Andrew Pinski - Re: [PATCH]: Add folding of builtin frexp (original) (raw)

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

• From: Andrew Pinski
• To: richard dot guenther at gmail dot com (Richard Guenther)
• Cc: ghazi at caip dot rutgers dot edu (Kaveh R. GHAZI), gcc-patches at gcc dot gnu dot org
• Date: Sun, 25 Feb 2007 19:10:41 -0500 (EST)
• Subject: Re: [PATCH]: Add folding of builtin frexp

On 2/22/07, Kaveh R. GHAZI ghazi@caip.rutgers.edu wrote:

This patch folds builtin frexp where thet first argument is constant.

Tested on sparc-sun-solaris2.10, no regressions.

Okay for mainline?

This is ok with the change below

            Thanks,
            --Kaveh

2007-02-21 Kaveh R. Ghazi ghazi@caip.rutgers.edu

    * builtins.c (fold_builtin_frexp): New.
    (fold_builtin_2): Use it.

testsuite: * gcc.dg/torture/builtin-frexp-1.c: New test.

I cannot find the final patch version but this testcase fail on powerpc-linux-gnu and powerpc64-linux-gnu. :builtin-frexp-1.c:(.text+0x720): undefined reference to link_error'^M :builtin-frexp-1.c:(.text+0x97c): undefined reference to link_error'^M :builtin-frexp-1.c:(.text+0xeb0): undefined reference to link_error'^M :builtin-frexp-1.c:(.text+0xff4): undefined reference to link_error'^M /tmp/ccQ7VUie.o:builtin-frexp-1.c:(.text+0x11e0): more undefined references to `link_error' follow^M

-- Pinski

• Follow-Ups:
  • Re: [PATCH]: Add folding of builtin frexp
    * From: Kaveh R. GHAZI
• References:
  • Re: [PATCH]: Add folding of builtin frexp
    * From: Richard Guenther