RFR: 8286990: Add compiler name to warning messages in Compiler Directive [v2]

Yuta Sato duke at openjdk.java.net
Tue May 31 03:27:41 UTC 2022


On Sun, 29 May 2022 00:03:42 GMT, Cesar Soares <duke at openjdk.java.net> wrote:

>> Yuta Sato has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update full name
>
> src/hotspot/share/compiler/compilerDirectives.hpp line 131:
> 
>> 129:   static ccstrlist canonicalize_control_intrinsic(ccstrlist option_value);
>> 130:   void finalize(outputStream* st);
>> 131:   bool is_c1(CompilerDirectives* directive);
> 
> NIT: these methods can be made "const".

@JohnTortugo 
Thank you for your advice.
I added "const" to these methods.

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

PR: https://git.openjdk.java.net/jdk/pull/8591


More information about the hotspot-compiler-dev mailing list