[jdk17u-dev] RFR: 8282347: AARCH64: Untaken branch in has_negatives stub [v3]

[Boris Ulasevich]

On Fri, 9 Sep 2022 08:52:53 GMT, Boris Ulasevich <bulasevich at openjdk.org> wrote:

>> Backport 3efd6aa4c9b6420520709281ab1ca46ba4242c87
>> 
>> A trivial change. Helps JDK-8281146 backport merge issues.
>
> Boris Ulasevich has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   Backport 3efd6aa4c9b6420520709281ab1ca46ba4242c87

Dear all,

My bad, I pushed the fix after PR review and completely forgot the review label mechanism. I am really sorry.

Let me ask for jdk17u-fix-request in correspondent JBS records records now. If the request is declined, I am ready to back out the changes.

I believe the changes I have pushed ([1] [2] [3]) are trivial, safe and harmless. The reason to bring them is a preparation for a big backport I want to propose - my goal is to bring the StringCoding.countPositives change [4] which has a dependencies:

+ commit 71ca0c08f09e87673f1cf3f918a0802961359166
+ 8270848: Redundant unsafe opmask register allocation in some instruction patterns.
|
| + commit 3efd6aa4c9b6420520709281ab1ca46ba4242c87
| + 8282347: AARCH64: Untaken branch in has_negatives stub
| |
| | + commit e314a4cfda30cc680b3f0aef8c62b75ff81bdbb1
| | + 8280124: Reduce branches decoding latin-1 chars from UTF-8 encoded bytes
| | |
| | |\
| | | + commit d4d12ad1d92ae6d353ce9e16b1b72e6416874cd6
| | | + 8282047: Enhance StringDecode/Encode microbenchmarks
 \ \ \ \
  +-+-+-+-+ commit beedae1141b6b650dc4cedf1f038afc1c8b460dd
          + 8281146: Replace StringCoding.hasNegatives with countPositives

[1] https://github.com/openjdk/jdk17u-dev/pull/657
[2] https://github.com/openjdk/jdk17u-dev/pull/658
[3] https://github.com/openjdk/jdk17u-dev/pull/659
[4] https://bugs.openjdk.org/browse/JDK-8281146

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

PR: https://git.openjdk.org/jdk17u-dev/pull/659