Integrated: 8256754: Deoptimization::revoke_for_object_deoptimization: stack processing start call is redundant
Richard Reingruber
rrich at openjdk.java.net
Mon Nov 30 08:39:56 UTC 2020
On Mon, 23 Nov 2020 11:40:49 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
> This is a XS clean-up of Deoptimization::revoke_for_object_deoptimization() which removes the StackWatermarkSet::start_processing() call.
>
> This is correct because all paths leading to revoke_for_object_deoptimization() are equipped with a KeepStackGCProcessedMark.
>
> Call Tree:
>
> StackWatermarkSet::start_processing(JavaThread *, enum StackWatermarkKind) : void
> Deoptimization::revoke_for_object_deoptimization(JavaThread *, frame, RegisterMap *, JavaThread *) : void
> Deoptimization::deoptimize_objects_internal(JavaThread *, GrowableArray<compiledVFrame *> *, bool &) : bool
> EscapeBarrier::deoptimize_objects_internal(JavaThread *, intptr_t *) : bool
> EscapeBarrier::deoptimize_objects_all_threads() : bool // has KeepStackGCProcessedMark
> EscapeBarrier::deoptimize_objects(intptr_t *) : bool
> EscapeBarrier::deoptimize_objects(int, int) : bool // has KeepStackGCProcessedMark
>
> Testing: hotspot_serviceability, jdk_svc, jdk_jdi, vmTestbase_nsk_jdi, vmTestbase_nsk_jvmti, vmTestbase_nsk_jdwp with -XX:+UseZGC.
This pull request has now been integrated.
Changeset: e77aed62
Author: Richard Reingruber <rrich at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/e77aed62
Stats: 24 lines in 4 files changed: 20 ins; 3 del; 1 mod
8256754: Deoptimization::revoke_for_object_deoptimization: stack processing start call is redundant
Reviewed-by: dlong, eosterlund
-------------
PR: https://git.openjdk.java.net/jdk/pull/1381
More information about the hotspot-dev
mailing list