RFR: 8256325: Remove duplicate asserts in PhaseMacroExpand::expand_macro_nodes [v2]

Tobias Hartmann thartmann at openjdk.java.net
Fri Nov 13 12:00:33 UTC 2020


On Fri, 13 Nov 2020 11:20:27 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Uppercase DEBUG_ONLY
>
> Looks pretty simple and good. 
> 
> While you are here, you could also capitalize the macro and drop redundant semicolon:
>     debug_only(int old_macro_count = C->macro_count(););
> to:
>     DEBUG_ONLY(int old_macro_count = C->macro_count();)

@shipilev, @cl4es thanks for the reviews. I've updated `debug_only` to `DEBUG_ONLY`.

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

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


More information about the hotspot-compiler-dev mailing list