[aarch64-port-dev ] RFR: 8156839 : aarch64: indexOf does not support CompactStrings

Andrew Haley aph at redhat.com
Thu May 12 13:52:21 UTC 2016


Hi,

On 05/12/2016 02:43 PM, Edward Nevill wrote:
> Please review the following webrev which adds support for
> CompactStrings to the indexOf intrinsic.
> 
> http://cr.openjdk.java.net/~enevill/8156839/webrev.01

Looks reasonable.

> This adds support for the UU and LL encodings. Do we also need
> support for LU and UL?

I'm not sure.  What happens if you try to compare a latin1 and a UTF16
string?  Do we get compilation failures, or just a slow compare?

> Also, I notice that StrComp doesn't seem to have support for
> CompactString either. Shall I do that also?
> 
> Tested with jtreg hotspot and langtools.

OK.  But note well: hotspot/hs is frozen for the JDK9 release.

Andrew.


More information about the aarch64-port-dev mailing list