-
Type:
Enhancement
-
Resolution: Unresolved
-
Priority:
P4
-
Affects Version/s: None
-
Component/s: hotspot
It seems that JDK-8215473 introduced a substitutability test for acmp, first guarded by a flag, and then the flag was removed in JDK-8227033. There are still some comments referring to the old implementation, which can be confusing. For example:
https://github.com/openjdk/valhalla/blob/81b25a08ddd081b699ff7d721c3fdb6f6291f9fc/test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestNewAcmp.java#L131-L133
Such comments should be fixed/removed to avoid confusion (as they would hint towards wrong results).
https://github.com/openjdk/valhalla/blob/81b25a08ddd081b699ff7d721c3fdb6f6291f9fc/test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestNewAcmp.java#L131-L133
Such comments should be fixed/removed to avoid confusion (as they would hint towards wrong results).
- relates to
-
JDK-8215473 [lworld] Support for substitutability checks in C2
-
- Resolved
-
-
JDK-8227033 [lworld] Remove ACmpOnValues command line option
-
- Resolved
-