String intrinsics defunct on arch != amd64 after 8141132: JEP 254: Compact Strings

[Andrew Dinn]

On 30/11/15 18:54, Volker Simonis wrote:
> Moreover, "UseSSE42Intrinsics" is clearly a architecture-dependant
> option. I already wondered that according to vm_version_aarch64.cpp it
> seems to exists on aarch64 (is this really true Andrew?). But it's
> surely not available on PowerPC, SPARC, ...

I assume you meant the other Andrew? Anyway, you get a 2nd Andrew for free.

It appears that this bogus set default = true for UseSSE42Intrinsics was
included in our JDK8 tree when Ed Nevill implemented a StrIndexOf rule
in the AArch64 ad file. I guess it was done precisely to get round the
problem that a StrIndexOf rule only gets used if this value
UseSSE42Intrinsics is set.

So, yes this needs cleaning up to allow StrIndexOf nodes to be
implemented in non-x86_64 cases (including aarch64) without having to
set a bogus flag.

regards,


Andrew Dinn
-----------