JDK-8344108 : [lworld] Merge of 8340214 causes "slice of address and input slice don't match" asserts
  • Type: Bug
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: repo-valhalla
  • Priority: P4
  • Status: Closed
  • Resolution: Not an Issue
  • Submitted: 2024-11-13
  • Updated: 2025-03-27
  • Resolved: 2025-03-27
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
repo-valhallaResolved
Related Reports
Relates :  
JDK-8341411 - C2: remove slice parameter from GraphKit::make_load() and GraphKit::store_to_memory()
Relates :  
JDK-8340214 - C2 compilation asserts with "no node with a side effect" in PhaseIdealLoop::try_sink_out_of_loop
Description
Found an issue when merging jdk-24+19, specifically 8340214 caused assertion while running "compiler/valhalla/inlinetypes/TestIntrinsics.java"

Merge work commenting out "slice of address and input slice don't match" asserts in graphKit.cpp until Valhalla code is adjusted.
Comments
This appears to no longer be an issue once JDK-8341411 was merged in which removed the `adr_idx` parameter on which the failing assert was performed. I'm therefore closing this as not an issue (anymore).
27-03-2025