JDK-8156839 : aarch64: String.indexOf intrinsic does not support CompactStrings
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 9
  • Priority: P4
  • Status: Resolved
  • Resolution: Duplicate
  • OS: linux
  • CPU: aarch64
  • Submitted: 2016-05-12
  • Updated: 2016-10-04
  • Resolved: 2016-10-04
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 9
9Resolved
Related Reports
Blocks :  
Duplicate :  
Description
The aarch64 indexOf intrinsics does not support CompactStrings.

It should at least support LL

Comments
Dup of 8157834.
04-10-2016

I believe this has already been pushed. See http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/6db90ea983f1 This was pushed to resolve https://bugs.openjdk.java.net/browse/JDK-8157834 So I believe this bug is duplicate and should be closed.
04-10-2016

Please, change "Proposed Integration Date" in Request FC exemption since June 30 is passed.
19-07-2016

Approved for JDK 9.
21-06-2016

This bug is not the last piece, there is also the following open issues https://bugs.openjdk.java.net/browse/JDK-8156943 (aarch64: String.compare intrinsic does not support CompactStrings) https://bugs.openjdk.java.net/browse/JDK-8157708 (aarch64: StrIndexOfChar intrinsic is not implemented) There is also HasNegatives which does not have a JIRA issue. The umbrella issue is https://bugs.openjdk.java.net/browse/JDK-8156861 (AArch64: JEP 254: Partially-implemented intrinsics) for which the fix will be to turn CompactStrings on once all the other components are in place.
08-06-2016

If that bug was the last missing piece we should have turned CompactStrings on. I can't see that in the repo.
08-06-2016

This has been fixed as a result of 8157834
08-06-2016

Request FC exemption Justification: more time is needed to add CompactStrings support to AARCH64 intrinsics. Risk: Low. Only AARCH64 code is changed Proposed Integration Date: June 30, 2016
23-05-2016