RFR: 8263591: Two C2 compiler phases with the name "after matching" [v3]

Tue Mar 16 13:12:11 UTC 2021

On Tue, 16 Mar 2021 13:09:30 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> [JDK-8223347](https://bugs.openjdk.java.net/browse/JDK-8223347) added a `PHASE_AFTER_MATCHING` which is equivalent to  `PHASE_MATCHING` but only printed at level 3 and before node verification: 
>> https://github.com/openjdk/jdk/blob/20297a1bebfe03c45344a713ec561c97888c4750/src/hotspot/share/opto/compile.cpp#L2757-L2769
>> 
>> I don't think there is any benefit in having both and therefore removed it together with some other dead code (`idx` is always `0`).
>> 
>> Thanks,
>> Tobias
>
> Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Removed PHASE_AFTER_MATCHING from enum

Marked as reviewed by chagedorn (Reviewer).

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

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