Integrated: Fix Zero builds

Aleksey Shipilev shade at openjdk.org
Wed Apr 5 07:58:54 UTC 2023


On Tue, 4 Apr 2023 19:15:17 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> Current Zero builds (including ones in GHA) fail like this:
> 
> 
> In file included from /Users/shipilev/Work/shipilev-shenandoah/src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp:28:
> In file included from /Users/shipilev/Work/shipilev-shenandoah/src/hotspot/share/gc/shenandoah/shenandoahCollectionSet.hpp:30:
> /Users/shipilev/Work/shipilev-shenandoah/src/hotspot/share/gc/shenandoah/shenandoahHeap.hpp:452:17: error: inline function 'ShenandoahHeap::get_young_evac_reserve' is not defined [-Werror,-Wundefined-inline]
>   inline size_t get_young_evac_reserve() const;
>                 ^
> /Users/shipilev/Work/shipilev-shenandoah/src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp:110:50: note: used here
>       size_t max_young_cset    = (size_t) (heap->get_young_evac_reserve() / ShenandoahEvacWaste);
> 
> 
> This is because with Zero, we don't have the transitive include to `shenandoahHeap.inline.hpp`, yet `shenandoahAdaptiveHeuristics.cpp` uses it. The fix is -- as usual -- to add the explicit include.
> 
> Additional testing:
>   - [x] macosx-aarch64-zero-fastdebug
>   - [x] GHA

This pull request has now been integrated.

Changeset: 3a03d7dd
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/shenandoah/commit/3a03d7dd921c0de13e8ace19bf11722055de9748
Stats:     1 line in 1 file changed: 1 ins; 0 del; 0 mod

Fix Zero builds

Reviewed-by: kdnilsen

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

PR: https://git.openjdk.org/shenandoah/pull/244


More information about the shenandoah-dev mailing list