RFR: 8255274: [PPC64, s390] wrong StringLatin1.indexOf version matched

[Richard Reingruber]

On Thu, 22 Oct 2020 21:56:54 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> PPC64 and s390 currently match indexOfChar_U also for StrIntrinsicNode::L. This leads to incorrect results of StringLatin1.indexOf and alreads breaks builds:
> Optimizing the exploded image
> Error occurred during initialization of boot layer
> 
> We need separate match rules for StrIntrinsicNode::U and StrIntrinsicNode::L.

Ok, I see. Would be a worthwhile enhancement to preserve information that allows for having just one instruction form.
You might want to change to format string to indicate which version is used. I leave it to your discretion.
Thanks again for fixing.

-------------

Marked as reviewed by rrich (Committer).

PR: https://git.openjdk.java.net/jdk/pull/820