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

Frederic Parain fparain at openjdk.java.net
Mon Mar 28 13:07:09 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

This pull request has now been integrated.

Changeset: 77e57c26
Author:    Frederic Parain <fparain at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/77e57c2684d913b01bf13f121a652419151b2817
Stats:     9 lines in 3 files changed: 2 ins; 3 del; 4 mod

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

Reviewed-by: thartmann

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

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



More information about the valhalla-dev mailing list