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