RFR: 8356813: Improve Mod(I|L)Node::Value [v6]

Emanuel Peter epeter at openjdk.org
Mon Aug 25 13:23:02 UTC 2025


On Mon, 25 Aug 2025 13:09:06 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Hannes Greule has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 18 additional commits since the last revision:
>> 
>>  - typos
>>  - Merge branch 'master' into improve-mod-value
>>  - Merge branch 'master' into improve-mod-value
>>  - simplify UB/cpu exception check
>>  - wording
>>  - Address more comments
>>  - Merge branch 'master' into improve-mod-value
>>  - Add randomized test
>>  - Use BasicType for shared implementation
>>  - Update ModL comment
>>  - ... and 8 more: https://git.openjdk.org/jdk/compare/9e98b6eb...11210414
>
> src/hotspot/share/opto/divnode.cpp line 1207:
> 
>> 1205:   const Type* t2 = phase->type(in2);
>> 1206:   if (t1 == Type::TOP) return Type::TOP;
>> 1207:   if (t2 == Type::TOP) return Type::TOP;
> 
> Suggestion:
> 
>   if (t1 == Type::TOP) { return Type::TOP; }
>   if (t2 == Type::TOP) { return Type::TOP; }
> 
> If we already touch the code, we should also fix the brackets.

Please fix it below as well.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25254#discussion_r2298064781


More information about the hotspot-compiler-dev mailing list