[lworld] Integrated: 8235914: [lworld] Profile acmp bytecode

Roland Westrelin roland at openjdk.java.net
Mon Oct 12 13:32:37 UTC 2020


On Fri, 11 Sep 2020 07:58:46 GMT, Roland Westrelin <roland at openjdk.org> wrote:

> This includes:
> - a new ProfileData structure to profile both inputs to acmp
> - profile collection at acmp in the interpreter on x86
> - profile collection at acmp in c1 generated code on x86
> - changes to c2's acmp implementation to leverage profiling (both existing profiling through type speculation and new
>   profile data at acmp)
> - small tweaks to the assembly code generated for acmp
> - a change to the implementation of LIRGenerator::profile_null_free_array() so it doesn't use a branch (which is
>   dangerous given the register allocator is not aware of branches added at the LIR level)
> - new tests
> 
> Profile collection happens unconditionally. Leveraging profiling at acmp is under UseACmpProfile which is false by
> default.

This pull request has now been integrated.

Changeset: 378279c9
Author:    Roland Westrelin <roland at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/378279c9
Stats:     1261 lines in 44 files changed: 1133 ins; 59 del; 69 mod

8235914: [lworld] Profile acmp bytecode

Reviewed-by: thartmann

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

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



More information about the valhalla-dev mailing list