RFR: 8273148: Shouldn't increment _ifop_count when IfOp is created

Dean Long dlong at openjdk.java.net
Tue Aug 31 00:15:30 UTC 2021


On Mon, 30 Aug 2021 16:16:10 GMT, Denghui Dong <ddong at openjdk.org> wrote:

> Hi,
> 
> Please help review this small fix.
> 
> IIUC, _ifop_count is incremented when IfOp is no need to create, but there is a wrong increment operation in the current implementation(CE_Eliminator::make_ifop).
> 
> Thanks,
> Denghui

The comment for int _ifop_count says "the number of IfOps successfully simplified", not "eliminated".  If you want to distinguish between simplified and eliminated, I think we need another counter.

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

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


More information about the hotspot-compiler-dev mailing list