RFR: 8322735: C2: minor improvements of bubble sort used in SuperWord::packset_sort [v2]

Denghui Dong ddong at openjdk.org
Tue Jan 9 05:26:46 UTC 2024


On Mon, 8 Jan 2024 20:59:53 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Denghui Dong has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update
>
> src/hotspot/share/opto/superword.cpp line 3526:
> 
>> 3524:       // only swap when we find something to swap
>> 3525:       if (alignment(q_low->at(0)) > alignment(q_i->at(0))) {
>> 3526:         Node_List* t = q_i;
> 
> Why you need this local `t`?

Good catch. Deleted.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17190#discussion_r1445638467


More information about the hotspot-compiler-dev mailing list