JDK-8319813 : Remove upper limit on number of compiler phases in phasetype.hpp
  • Type: Enhancement
  • Component: hotspot
  • Sub-Component: compiler
  • Affected Version: 22
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2023-11-09
  • Updated: 2023-11-27
  • Resolved: 2023-11-23
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.
JDK 22
22 masterFixed
Related Reports
Blocks :  
JDK-8295166 - IGV: dump graph at more locations
Description
The field `uint64_t _ideal_phase_name_mask;` in DirectiveSet (used to store compiler directives) implicitly limits the number of phases in phasetype.hpp to 64. With the recent work on JDK-8295166, we are now surpassing this upper bound and need to remove it.

Compile::should_print_phase is the only function that makes use of _ideal_phase_name_mask.
Comments
Changeset: 8db7bad9 Author: Daniel Lundén <daniel.lunden@oracle.com> Committer: Emanuel Peter <epeter@openjdk.org> Date: 2023-11-23 08:42:09 +0000 URL: https://git.openjdk.org/jdk/commit/8db7bad992a0f31de9c7e00c2657c18670539102
23-11-2023
A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/16729 Date: 2023-11-20 09:04:27 +0000
20-11-2023