RFR (S): 8181787: Remove Solaris SPARC GCC code
coleen.phillimore at oracle.com
coleen.phillimore at oracle.com
Thu Jun 8 16:59:22 UTC 2017
Erik, Looks good. It was an ancient experiment to enable case labels
in bytecodeInterpreter.cpp on solaris.
thanks,
Coleen
On 6/8/17 11:06 AM, Erik Österlund wrote:
> Hi,
>
> The only supported way of building OpenJDK on Solaris is via the
> studio compiler. However, there are unsupported code paths that are
> enabled only if _GNU_SOURCE is defined. This might be problematic for
> multiple reasons. Checking for _GNU_SOURCE does not check for GCC, it
> checks whether the standard library is the GNU standard library. Most
> importantly, the code has been bit rotting and e.g. the inline
> assembly for atomic seems to have flaws. Since we never test the
> correctness of these code paths, I propose to remove this unsupported
> code.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8181787
>
> Webrev: http://cr.openjdk.java.net/~eosterlund/8181787/webrev.00/
>
> Testing: JPRT
>
> Will need a sponsor.
>
> Thanks,
> /Erik
More information about the hotspot-runtime-dev
mailing list