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

Vladimir Kozlov kvn at openjdk.org
Fri Jan 10 20:42:42 UTC 2025


On Fri, 10 Jan 2025 14:02:32 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.
>
> Damon Fenacci has updated the pull request incrementally with one additional commit since the last revision:
> 
>   JDK-8347407: re-fix JBS issue number in problem list file

Good

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

Marked as reviewed by kvn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/23031#pullrequestreview-2543771293


More information about the hotspot-compiler-dev mailing list