Integrated: 8265322: C2: Simplify control inputs for BarrierSetC2::obj_allocate
Yi Yang
yyang at openjdk.java.net
Mon May 3 14:26:52 UTC 2021
On Fri, 16 Apr 2021 03:06:19 GMT, Yi Yang <yyang at openjdk.org> wrote:
> While just using BarrierSetC2::obj_allocate in https://github.com/openjdk/valhalla/pull/385, I noticed the control input `ctrl` for obj_allocate is not so much necessary. This PR simplifies control inputs for BarrierSetC2::obj_allocate. In most cases, it doesn't change anything since `toobig_false` is equivalent to `ctrl`. In other cases, `toobig_false` is created for Unsafe.allocateInstance while instance size is not statically known, `ctrl` would become control input of IfNode whose projects are `toobig_false` and `toobig_true`, old eden_end and old_eden_top can simply accept `toobig_false` as their control input rather than `ctrl`.
This pull request has now been integrated.
Changeset: 001c5142
Author: Yi Yang <yyang at openjdk.org>
Committer: Nils Eliasson <neliasso at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/001c5142a6ff4c4073e651ebae9d6d7a8533eb42
Stats: 5 lines in 3 files changed: 0 ins; 0 del; 5 mod
8265322: C2: Simplify control inputs for BarrierSetC2::obj_allocate
Reviewed-by: kvn, neliasso
-------------
PR: https://git.openjdk.java.net/jdk/pull/3529
More information about the hotspot-gc-dev
mailing list