RFR: 8325144: C1: Optimize CriticalEdgeFinder

[Dean Long]

On Thu, 1 Feb 2024 16:49:02 GMT, Denghui Dong <ddong at openjdk.org> wrote:

> Hi,
> 
> Please help review this change that removed the sorting process in split_edges by checking if the 'to' block has been substituted.
> 
> Thanks

src/hotspot/share/c1/c1_IR.cpp line 324:

> 322: 
> 323:  public:
> 324:   CriticalEdgeFinder(IR* ir): _ir(ir) {}

Yes, we can remove the ctor, but wouldn't it be better to keep the ctor to initialize `blocks`?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17674#discussion_r1487087511