RFR: 8324630: C1: Canonicalizer::do_LookupSwitch doesn't break the loop when the successor is found [v3]
Tobias Hartmann
thartmann at openjdk.org
Mon Feb 19 08:25:54 UTC 2024
On Fri, 26 Jan 2024 22:47:46 GMT, Denghui Dong <ddong at openjdk.org> wrote:
>> Hi,
>>
>> Please review the small change that breaks the loop in Canonicalizer::do_LookupSwitch if the successor is found.
>>
>> The keys of LookupSwitch are sorted, so there is no need to continue the loop once matched.
>>
>> Thanks.
>
> Denghui Dong has updated the pull request incrementally with one additional commit since the last revision:
>
> binary search
Looks good to me.
-------------
Marked as reviewed by thartmann (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/17553#pullrequestreview-1887807893
More information about the hotspot-compiler-dev
mailing list