There are two instances within harfbuzz where the conditional checks seem impossible or redundant. I have removed these redundant checks and tested the changes. Everything still works as expected.
The !last_range check is logically redundant because there is the other check if (unlikely (last_range)) beforehand in the same segment. This is effectively similar to a null check. The next check for peak != 0 is also redundant. This is because of if (peak == 0 || coord == peak) earlier in the code that returns 1.f. So all following instructions should logically be when peak is not 0. ------------- Commit messages: - Initial commit - Merge branch 'master' of https://github.com/openjdk/jdk - Initial commit Changes: https://git.openjdk.org/jdk/pull/30412/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=30412&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8380229 Stats: 3 lines in 2 files changed: 0 ins; 1 del; 2 mod Patch: https://git.openjdk.org/jdk/pull/30412.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/30412/head:pull/30412 PR: https://git.openjdk.org/jdk/pull/30412
