JDK-8209686 : cleanup arguments to PhaseIdealLoop() constructor
Type:Enhancement
Component:hotspot
Sub-Component:compiler
Affected Version:9,10,11,12
Priority:P4
Status:Resolved
Resolution:Fixed
Submitted:2018-08-20
Updated:2020-04-03
Resolved:2018-08-22
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.
PhaseIdealLoop() is passed 3 boolean arguments to define what level of optimizations to apply. I propose using a single enum argument instead.
Comments
Fix-request (11u): This change is a pre-requisite for the Shenandoah/jdk11u backport, it makes that one much less exposed in shared-code. It's also a nice and rather harmless cleanup.
Review-thread:
https://mail.openjdk.java.net/pipermail/jdk-updates-dev/2020-February/002463.html
Testing: tier1 and tier2 tests are clean