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

Fri Nov 13 12:00:32 UTC 2020

On Fri, 13 Nov 2020 11:57:52 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> [JDK-8239072](https://bugs.openjdk.java.net/browse/JDK-8239072) added a default case to the switch statement in PhaseMacroExpand::expand_macro_nodes:
>> https://hg.openjdk.java.net/jdk/jdk/rev/cf319f17c647#l3.67
>> 
>> This allows to merge the "expansion must have deleted one node from macro list" asserts and remove the duplicate asserts.
>> 
>> Thanks,
>> Tobias
>
> Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Uppercase DEBUG_ONLY

Looks good and trivial.

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

Marked as reviewed by redestad (Reviewer).

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