On Fri, 3 Oct 2025 22:12:19 GMT, Joe Darcy <[email protected]> wrote:
>> Upon further inspection, found a few more opportunities to refine the use of >> constants in the Float16 implementation. > > Joe Darcy has updated the pull request incrementally with one additional > commit since the last revision: > > Remove unneeded use of "Float16.". > > ``` > > // IEEEremainder / remainder operator remainder > > ``` > `IEEEremainder` differs from the Java language remainder operator, due to the > difference in rounding policy. Thanks for providing context, it makes the sentence easier to understand. I now parse "remainder operator remainder" as "the remainder function as defined by JVM drem/frem bytecodes". The comment seems a bit terse in its current form, more of a note-to-self reminder of future implementation ideas. Usually comments refer to current functionality, so this was sticking out to me as somewhat unusual. @jddarcy may have an opinion here. ------------- PR Comment: https://git.openjdk.org/jdk/pull/27625#issuecomment-3368894135
