RFR: 8319813: Remove upper limit on number of compiler phases in phasetype.hpp [v3]

Daniel Lundén duke at openjdk.org
Tue Nov 21 09:48:11 UTC 2023


On Tue, 21 Nov 2023 07:51:02 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Inline single-use function for empty check
>
> src/hotspot/share/compiler/compilerDirectives.cpp line 622:
> 
>> 620: 
>> 621:   set->_intrinsic_control_words = src->_intrinsic_control_words;
>> 622:   set->_ideal_phase_name_set.set_from(src->_ideal_phase_name_set);
> 
> The advantage of passing the set/mask directly is that here you could also use your `set_ideal_phase_name_set` method.

Thanks, updated.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16729#discussion_r1400295367


More information about the hotspot-compiler-dev mailing list