RFR: 8319822: Use a linear-time algorithm for assert_different_registers() [v6]

Andrew Haley aph at openjdk.org
Fri May 10 15:05:23 UTC 2024


On Wed, 17 Jan 2024 07:10:13 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Andrew Haley has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Review feedback.
>
> src/hotspot/cpu/aarch64/register_aarch64.hpp line 73:
> 
>> 71: 
>> 72:   constexpr bool operator==(const Register r) const { return _encoding == r._encoding; }
>> 73:   constexpr bool operator!=(const Register r) const { return _encoding != r._encoding; }
> 
> This seems unrelated to the rest of this change.  It also seems like something that should be done for all
> of the register_<cpu> variants.

It was related to another reviewer's comments, but we don't need it

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16617#discussion_r1596871037


More information about the hotspot-compiler-dev mailing list