On Tue, 1 Mar 2022 18:32:00 GMT, Vladimir Kozlov <k...@openjdk.org> wrote:

> @theRealAph , @a74nh or someone familiar with aarch64 code, please review 
> aarch64 changes.

Note that the aarch64 changes I've put in for now implements `countPositives` 
to return `0` if there's a negative value anywhere, otherwise `len`. This way 
we can remove the intrinsic scaffolding for `hasNegatives` once I integrate 
s390 and ppc changes, but since no precise calculation is implemented for 
aarch64 then there's no speed-up on micros such as `encodeLatin1LongEnd` for 
now. I would like to defer such optimization work to a follow-up.

@RealLucy yes, I'd like to integrate your changes into this patch and drop the 
`HasNegativesNode` et.c. You mentioned in e-mail some issue with the code 
generation on s390 - has that been resolved or identified?

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

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

Reply via email to