diff options
| author | Tom Tromey <tromey@redhat.com> | 2006-02-04 23:34:06 +0000 |
|---|---|---|
| committer | Tom Tromey <tromey@gcc.gnu.org> | 2006-02-04 23:34:06 +0000 |
| commit | 05d8200d79f284ba5cf5e552183fae438e4f3bf7 (patch) | |
| tree | 729c397263390e58a1e0056350bf2b1959af7c43 /gcc/java/builtins.c | |
| parent | fb579387f96f84e0d1e3a9bc5baded6d549824b8 (diff) | |
| download | gcc-05d8200d79f284ba5cf5e552183fae438e4f3bf7.zip gcc-05d8200d79f284ba5cf5e552183fae438e4f3bf7.tar.gz gcc-05d8200d79f284ba5cf5e552183fae438e4f3bf7.tar.bz2 | |
re PR java/25676 (Use of MIN_EXPR/MAX_EXPR is wrong for java)
gcc/java
PR java/25676:
* builtins.c (max_builtin): Skip floating point 'max'.
(min_builtin): Skip floating point 'min'.
(check_for_builtin): Never return NULL_TREE.
libjava
PR java/25676:
* testsuite/libjava.lang/pr25676.out: New file.
* testsuite/libjava.lang/pr25676.java: New file.
From-SVN: r110599
Diffstat (limited to 'gcc/java/builtins.c')
| -rw-r--r-- | gcc/java/builtins.c | 18 |
1 files changed, 14 insertions, 4 deletions
diff --git a/gcc/java/builtins.c b/gcc/java/builtins.c index ff6da98..5082cd0 100644 --- a/gcc/java/builtins.c +++ b/gcc/java/builtins.c @@ -1,5 +1,5 @@ /* Built-in and inline functions for gcj - Copyright (C) 2001, 2003, 2004, 2005 + Copyright (C) 2001, 2003, 2004, 2005, 2006 Free Software Foundation, Inc. This file is part of GCC. @@ -94,6 +94,9 @@ static GTY(()) struct builtin_record java_builtins[] = static tree max_builtin (tree method_return_type, tree method_arguments) { + /* MAX_EXPR does not handle -0.0 in the Java style. */ + if (TREE_CODE (method_return_type) == REAL_TYPE) + return NULL_TREE; return fold_build2 (MAX_EXPR, method_return_type, TREE_VALUE (method_arguments), TREE_VALUE (TREE_CHAIN (method_arguments))); @@ -102,6 +105,9 @@ max_builtin (tree method_return_type, tree method_arguments) static tree min_builtin (tree method_return_type, tree method_arguments) { + /* MIN_EXPR does not handle -0.0 in the Java style. */ + if (TREE_CODE (method_return_type) == REAL_TYPE) + return NULL_TREE; return fold_build2 (MIN_EXPR, method_return_type, TREE_VALUE (method_arguments), TREE_VALUE (TREE_CHAIN (method_arguments))); @@ -265,11 +271,15 @@ check_for_builtin (tree method, tree call) tree fn; if (java_builtins[i].creator != NULL) - return (*java_builtins[i].creator) (method_return_type, - method_arguments); + { + tree result + = (*java_builtins[i].creator) (method_return_type, + method_arguments); + return result == NULL_TREE ? call : result; + } fn = built_in_decls[java_builtins[i].builtin_code]; if (fn == NULL_TREE) - return NULL_TREE; + return call; return java_build_function_call_expr (fn, method_arguments); } } |