RFR(M): 8215902: Add support for SoftFloat-3e library
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Tue Jan 15 14:05:23 UTC 2019
On 2018-12-25 16:19, Jakub Vaněk wrote:
> Hi,
>
> please review this webrev. It is a successor of the softfloat-3 [patch]
> thread (first email
> http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2018-November/031311.html
> )
>
> Changes since the last patch (v6):
>
> - renamed --with-softloat* to --with-sflt* (it is more compact and it
> corresponds to the old --with-sflt-lib=... option)
>
> - license is now obtained via --with-sflt-license switch (so it is not
> included in OpenJDK source tree)
>
> - updated documentation (slight rewording, added the license option)
>
> - checks for default --with/--without behavior are in place again
> (I forgot them when I changed the way the library is detected)
>
> - added a simple testcase - I found a disrepancy between softfloat and
> system function behavior. When a float with bits 0x003FFFFF is
> added to 0x00000001, the correct result is 0x00400000, but the
> default software floating point implementation returns 0x00000000.
> However I'm not sure where to put this test - now it is in
> test/hotspot/jtreg/compiler/floatingpoint.
>
> - comments in code refer to CR 6757269 and newly JDK-8215902 too.
>
> I have created a repository with SoftFloat-3e with build configuration
> specifically for OpenJDK on armel:
> https://github.com/ev3dev-lang-java/softfloat-openjdk
>
> I can add a link to it to the documentation.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8215902
> Webrev: http://cr.openjdk.java.net/~jakvanek/8215902/webrev.02/
Hi Jakub,
In general this looks good.
Some comments:
I agree with Erik that you can add a link to your github project;
compiling SoftFloat is outside the scope of the OpenJDK build
instructions, but it can sure be helpful to lower the bar for users
wanting to do that. Just one question: any particular reason you didn't
create your github repo by forking the official
https://github.com/ucb-bar/berkeley-softfloat-3? That way, it would have
been easy for users to see that you were not adding any malicious or
suspicious code to the original SoftFloat distribution.
On the other hand, I think the link to
http://mail.openjdk.java.net/pipermail/aarch32-port-dev/2016-November/000611.html
is unnecessary and just creates clutter in the documentation. Please
remove it.
/Magnus
> CI build: https://ci.adoptopenjdk.net/view/ev3dev/job/openjdk12_build_ev3_linux/67/
>
> Cheers,
>
> Jakub
>
More information about the hotspot-compiler-dev
mailing list