Integrated: 8282792: Loom: Fix optimized build after commit 4d2710

Hao Sun haosun at openjdk.java.net
Thu Mar 24 09:23:57 UTC 2022


On Wed, 9 Mar 2022 06:48:49 GMT, Hao Sun <haosun at openjdk.org> wrote:

> In commit 4d2710[1], one invocation of function print_frames() is moved
> from condition `!PRODUCT` to condition `ASSERT`. As a result, all the
> usages are now under condition `ASSERT`, leading to the compilation
> warning "unused-function" in optimized VM build.
> 
> Hence, in this patch, we move the defintion of function print_frames()
> to condition `ASSERT`.
> 
> [1] https://github.com/openjdk/loom/commit/4d2710#diff-74c5a41cb3476b29151b4a13732cf4756879de0aa1b1d8c4d73c38e570413fcfR2851

This pull request has now been integrated.

Changeset: b83c1c82
Author:    Hao Sun <haosun at openjdk.org>
Committer: Erik Österlund <eosterlund at openjdk.org>
URL:       https://git.openjdk.java.net/loom/commit/b83c1c828962ff274daf26b9b8ff2802a255a2a3
Stats:     52 lines in 1 file changed: 25 ins; 27 del; 0 mod

8282792: Loom: Fix optimized build after commit 4d2710

Reviewed-by: eosterlund

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

PR: https://git.openjdk.java.net/loom/pull/98


More information about the loom-dev mailing list