Integrated: 8274328: C2: Redundant CFG edges fixup in block ordering

[Yi Yang]

On Sun, 26 Sep 2021 10:40:43 GMT, Yi Yang <yyang at openjdk.org> wrote:

> I think Trace::fixup_blocks is redundant because PhaseCFG::fixup_flow will nevertheless fix up the CFG flow(i.e. flip successor blocks of IfNode) right after PhaseBlockLayout pass, we can remove this step when doing PhaseBlockLayout pass.(Testing: jtreg/compiler/c2, presubmit test)
> 
> https://github.com/openjdk/jdk/blob/5ec1cdcaf39229a7d2457313600b0dc2bf8c6453/src/hotspot/share/opto/compile.cpp#L2765
> 
> https://github.com/openjdk/jdk/blob/5ec1cdcaf39229a7d2457313600b0dc2bf8c6453/src/hotspot/share/opto/block.cpp#L1679
> 
> https://github.com/openjdk/jdk/blob/5ec1cdcaf39229a7d2457313600b0dc2bf8c6453/src/hotspot/share/opto/block.cpp#L908-L916

This pull request has now been integrated.

Changeset: 44047f84
Author:    Yi Yang <yyang at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/44047f849fad157dac5df788aa5a2c1838e4aaf7
Stats:     56 lines in 2 files changed: 6 ins; 45 del; 5 mod

8274328: C2: Redundant CFG edges fixup in block ordering

Reviewed-by: thartmann, kvn

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

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