public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug java/22578] New: should inline floatToIntBits et al
@ 2005-07-20 20:41 tromey at gcc dot gnu dot org
2005-07-20 20:42 ` [Bug java/22578] " pinskia at gcc dot gnu dot org
0 siblings, 1 reply; 5+ messages in thread
From: tromey at gcc dot gnu dot org @ 2005-07-20 20:41 UTC (permalink / raw)
To: gcc-bugs
Currently Float and Double diverge from classpath.
I think we could safely inline these methods wherever they are called,
and then fully merge with Classpath. The result would not appreciably
affect performance, I think (and these aren't called much anyway).
--
Summary: should inline floatToIntBits et al
Product: gcc
Version: 4.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P2
Component: java
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: tromey at gcc dot gnu dot org
CC: gcc-bugs at gcc dot gnu dot org,java-prs at gcc dot gnu
dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22578
^ permalink raw reply [flat|nested] 5+ messages in thread* [Bug java/22578] should inline floatToIntBits et al 2005-07-20 20:41 [Bug java/22578] New: should inline floatToIntBits et al tromey at gcc dot gnu dot org @ 2005-07-20 20:42 ` pinskia at gcc dot gnu dot org 0 siblings, 0 replies; 5+ messages in thread From: pinskia at gcc dot gnu dot org @ 2005-07-20 20:42 UTC (permalink / raw) To: gcc-bugs ------- Additional Comments From pinskia at gcc dot gnu dot org 2005-07-20 20:42 ------- Confirmed, make sure they get inlined using either VIEW_CONVERT_EXPR or using an union, otherwise aliasing problems will show up. -- What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |enhancement Status|UNCONFIRMED |NEW Ever Confirmed| |1 Keywords| |missed-optimization Last reconfirmed|0000-00-00 00:00:00 |2005-07-20 20:42:20 date| | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22578 ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <bug-22578-360@http.gcc.gnu.org/bugzilla/>]
* [Bug java/22578] should inline floatToIntBits et al [not found] <bug-22578-360@http.gcc.gnu.org/bugzilla/> @ 2006-02-07 20:12 ` tromey at gcc dot gnu dot org 2006-02-08 18:06 ` tromey at gcc dot gnu dot org 2006-02-08 18:08 ` tromey at gcc dot gnu dot org 2 siblings, 0 replies; 5+ messages in thread From: tromey at gcc dot gnu dot org @ 2006-02-07 20:12 UTC (permalink / raw) To: gcc-bugs ------- Comment #2 from tromey at gcc dot gnu dot org 2006-02-07 20:12 ------- Testing a patch -- tromey at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|unassigned at gcc dot gnu |tromey at gcc dot gnu dot |dot org |org Status|NEW |ASSIGNED Last reconfirmed|2006-01-15 20:04:50 |2006-02-07 20:12:31 date| | http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22578 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug java/22578] should inline floatToIntBits et al [not found] <bug-22578-360@http.gcc.gnu.org/bugzilla/> 2006-02-07 20:12 ` tromey at gcc dot gnu dot org @ 2006-02-08 18:06 ` tromey at gcc dot gnu dot org 2006-02-08 18:08 ` tromey at gcc dot gnu dot org 2 siblings, 0 replies; 5+ messages in thread From: tromey at gcc dot gnu dot org @ 2006-02-08 18:06 UTC (permalink / raw) To: gcc-bugs ------- Comment #3 from tromey at gcc dot gnu dot org 2006-02-08 18:06 ------- Subject: Bug 22578 Author: tromey Date: Wed Feb 8 18:06:11 2006 New Revision: 110759 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110759 Log: gcc/java PR java/22578: * check-init.c (check_init): Handle VIEW_CONVERT_EXPR. * builtins.c (convert_real): New function. (java_builtins): Handle Float.intBitsToFloat, Float.floatToRawIntBits, Double.longBitsToDouble, Double.doubleToRawLongBits. libjava PR java/22578: * gcj/javaprims.h: Updated. * sources.am, Makefile.in: Rebuilt. * java/lang/natDouble.cc (doubleToLongBits): Moved to VMDouble. (doubleToRawLongBits): Likewise. (longBitsToDouble): Likewise. (toString): Likewise. (parseDouble): Likewise. * java/lang/natFloat.cc (floatToIntBits): Moved to VMFloat. (floatToRawIntBits): Likewise. (intBitsToFloat): Likewise. * java/lang/VMDouble.java: New file. * java/lang/VMFloat.java: New file. * java/lang/Float.java, java/lang/Double.java: Removed. Added: trunk/libjava/java/lang/VMDouble.java trunk/libjava/java/lang/VMFloat.java Removed: trunk/libjava/java/lang/Double.java trunk/libjava/java/lang/Float.java Modified: trunk/gcc/java/ChangeLog trunk/gcc/java/builtins.c trunk/gcc/java/check-init.c trunk/libjava/ChangeLog trunk/libjava/Makefile.in trunk/libjava/gcj/javaprims.h trunk/libjava/java/lang/natDouble.cc trunk/libjava/java/lang/natFloat.cc trunk/libjava/sources.am -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22578 ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Bug java/22578] should inline floatToIntBits et al [not found] <bug-22578-360@http.gcc.gnu.org/bugzilla/> 2006-02-07 20:12 ` tromey at gcc dot gnu dot org 2006-02-08 18:06 ` tromey at gcc dot gnu dot org @ 2006-02-08 18:08 ` tromey at gcc dot gnu dot org 2 siblings, 0 replies; 5+ messages in thread From: tromey at gcc dot gnu dot org @ 2006-02-08 18:08 UTC (permalink / raw) To: gcc-bugs ------- Comment #4 from tromey at gcc dot gnu dot org 2006-02-08 18:08 ------- Fix checked in. -- tromey at gcc dot gnu dot org changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED Target Milestone|--- |4.2.0 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=22578 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2006-02-08 18:08 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-07-20 20:41 [Bug java/22578] New: should inline floatToIntBits et al tromey at gcc dot gnu dot org
2005-07-20 20:42 ` [Bug java/22578] " pinskia at gcc dot gnu dot org
[not found] <bug-22578-360@http.gcc.gnu.org/bugzilla/>
2006-02-07 20:12 ` tromey at gcc dot gnu dot org
2006-02-08 18:06 ` tromey at gcc dot gnu dot org
2006-02-08 18:08 ` tromey at gcc dot gnu dot org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).