<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