On Wed, 5 Feb 2025 10:20:11 GMT, Per Minborg <pminb...@openjdk.org> wrote:
>> This PR proposes to add some `@ForceInline` annotations in the `Module` >> class in order to assist inlining of FFM var/method handles. >> >> Tested and passed tier1-3 > > src/java.base/share/classes/java/lang/Module.java line 488: > >> 486: // if other is an unnamed module then check if this module reads >> 487: // all unnamed modules >> 488: return !other.isNamed() > > This is a drive-by simplification. Can you revert this, the original code was easier to read. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23460#discussion_r1943023700