JDK-8156839 : aarch64: String.indexOf intrinsic does not support CompactStrings
Type:Enhancement
Component:hotspot
Sub-Component:compiler
Affected Version:9
Priority:P4
Status:Closed
Resolution:Duplicate
OS:linux
CPU:aarch64
Submitted:2016-05-12
Updated:2023-07-21
Resolved:2023-07-21
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.
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