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.
Related Reports
Blocks :  
Duplicate :  
The aarch64 indexOf intrinsics does not support CompactStrings.

It should at least support LL

Dup of 8157834.

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.

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

Approved for JDK 9.

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.

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

This has been fixed as a result of 8157834

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