[lworld] RFR: 8265121: [lworld] Better optimize null-checks in C1

Frederic Parain fparain at openjdk.java.net
Mon Mar 28 13:07:08 UTC 2022


On Fri, 25 Mar 2022 13:42:40 GMT, Frederic Parain <fparain at openjdk.org> wrote:

> Please review those small changes in C1 that eliminates a few more unnecessary null checks.
> I've experimented some changes in the NullCheckEliminator to handle inline types explicitly,
> but it had no effect on generated code, in all cases, existing null check eliminations were already
> sufficient to remove unnecessary checks.
> 
> The changeset also reverts a change in the CFG file generation which makes log file unreadable
> by the C1Visualizer tool.
> 
> Tested with Mach5, tier1-3.
> 
> Thank you.
> 
> Fred

Tobias,

Thank you for the review. I'll file a CR to fix the CFG file bug in mainline.

Fred

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

PR: https://git.openjdk.java.net/valhalla/pull/672



More information about the valhalla-dev mailing list