<i18n dev> RFR: 8263091: Remove CharacterData.isOtherUppercase/-Lowercase

Claes Redestad redestad at openjdk.java.net
Fri Mar 5 16:02:29 UTC 2021

This patch removes the CharacterData.isOtherUppercase and isOtherLowercase methods. It also exploits the fact that isOtherUppercase is always false for all codepoints in the CharacterDataLatin1 range for a small speed-up.

I have no means to test if this is correct on PPC, which has intrinsics for isLowerCase/isUpperCase, but unless I'm reading the code wrong the intrinsic for isLowerCase on PPC already appears to effectively do the fused logic of isLowerCase(ch) || isOtherLowerCase(ch) since it handles the two values where isLowerCase and isOtherLowercase disagrees (0xaa, 0xba), which means this change should make the intrinsic and the java code be in better agreement.


Commit messages:
 - Fold CharacterData.isOtherUpper-/Lowercase into isUpper-/LowerCase

Changes: https://git.openjdk.java.net/jdk/pull/2846/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2846&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8263091
  Stats: 98 lines in 8 files changed: 1 ins; 71 del; 26 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2846.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2846/head:pull/2846

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

More information about the i18n-dev mailing list