A random Java developer here passing by... great to see work on HarfBuzz performance! Thanks for your efforts to push JetBrains JDK improvements upstream.
I wonder if this patch might also fix or improve https://bugs.openjdk.java.net/browse/JDK-8202131 ? In case it's of any use, I have a ready-made performance test for this at https://gist.github.com/eirikbakke/81eec72221c9d7d8fc0b4afa9c54a163 . -- Eirik From: 2d-dev <2d-dev-boun...@openjdk.java.net> On Behalf Of Dmitry Batrak Sent: Wednesday, March 6, 2019 7:15 AM To: 2d-dev <2d-dev@openjdk.java.net> Subject: [OpenJDK 2D-Dev] [PATCH] 8220231: Cache HarfBuzz face object for same font's text layout calls Hello, I'd like to submit a patch for JDK-8220231. I'm not a Committer, so I'll need someone to sponsor this change. The proposed approach is used without known issues in OpenJDK-based JetBrains Runtime for almost three years now. I've mentioned it previously on this mailing list (https://mail.openjdk.java.net/pipermail/2d-dev/2017-August/008497.html). The change has been refactored as compared to the version mentioned above (the logic has been moved to SunLayoutEngine), and includes the removal of font tables caching (JDK-8186317). The latter, I believe, becomes redundant with this fix. Issue: https://bugs.openjdk.java.net/browse/JDK-8220231 Webrev: http://cr.openjdk.java.net/~dbatrak/8220231/webrev.00/ Best regards, Dmitry Batrak
