Integrated: 8347407: [BACKOUT] C1/C2 don't handle allocation failure properly during initialization (RuntimeStub::new_runtime_stub fatal crash)

Damon Fenacci dfenacci at openjdk.org
Mon Jan 13 08:02:41 UTC 2025


On Fri, 10 Jan 2025 09:34:19 GMT, Damon Fenacci <dfenacci at openjdk.org> wrote:

> This reverts _8326615: C1/C2 don't handle allocation failure properly during initialization (RuntimeStub::new_runtime_stub fatal crash)_ (commit 633fad8).
> The fix increased the code cache but was incomplete and didn't fix the underlying issue with code cache allocation potentially crashing (for the latter we have JBS issue [JDK-8339700](https://bugs.openjdk.org/browse/JDK-8339700)).
> 
> The `compiler/compilerDirectives.hpp` and `ci/ciStreams.hpp` were removed by [JDK-8345801](https://bugs.openjdk.org/browse/JDK-8345801) because the symbols needed were imported by `#include "c1/c1_Compiler.hpp"`. By removing this we need to put the 2 includes back.

This pull request has now been integrated.

Changeset: b37f1236
Author:    Damon Fenacci <dfenacci at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b37f12362507fb2cd291a2b44b4777ba76efd35e
Stats:     44 lines in 9 files changed: 5 ins; 26 del; 13 mod

8347407: [BACKOUT] C1/C2 don't handle allocation failure properly during initialization (RuntimeStub::new_runtime_stub fatal crash)

Reviewed-by: thartmann, kvn

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

PR: https://git.openjdk.org/jdk/pull/23031


More information about the hotspot-compiler-dev mailing list