On Thu, 28 May 2026 17:08:38 GMT, Manuel Hässig <[email protected]> wrote:

> This is a surprisingly clean cherry-pick of the JVMCI removal from mainline 
> with minor additional Valhalla related cleanups. The few conflicts were 
> mostly trivial to resolve. The non-tivial conflicts were in 
> `src/hotspot/cpu/aarch64//gc/shared/barrierSetNMethod_aarch64.cpp`, and 
> `src/hotspot/share/runtime/deoptimization.*`.
> 
> The full list of conflicts:
> 
> src/hotspot/cpu/aarch64/gc/shared/barrierSetNMethod_aarch64.cpp
> src/hotspot/cpu/aarch64/sharedRuntime_aarch64.cpp
> src/hotspot/cpu/x86/macroAssembler_x86.cpp
> src/hotspot/cpu/x86/sharedRuntime_x86_64.cpp
> src/hotspot/cpu/x86/templateInterpreterGenerator_x86.cpp
> src/hotspot/share/code/nmethod.cpp
> src/hotspot/share/compiler/compileBroker.cpp
> src/hotspot/share/oops/fieldInfo.hpp
> src/hotspot/share/oops/methodData.hpp
> src/hotspot/share/runtime/deoptimization.cpp
> src/hotspot/share/runtime/deoptimization.hpp
> src/java.base/share/classes/module-info.java
> test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java
> test/hotspot/jtreg/runtime/cds/serviceability/ReplaceCriticalClassesForSubgraphs.java
> test/hotspot/jtreg/runtime/clinit/ClassInitBarrier.java
> 
> 
> Testing:
>  - [ ] tier1,tier2,tier3,valhalla-comp-stress
>  - [ ] tier1,tier2,tier3,tier4,tier5 with `--enable-preview`
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

There was one thing I missed: the deleted 
`INTRINSIC_OR_TYPE_CHECKED_INLINING_TRAP` IR rule had a use in a Valhalla test. 
These were converted to the `INTRINSIC_TRAP` rule.

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

PR Comment: https://git.openjdk.org/valhalla/pull/2482#issuecomment-4571863161

Reply via email to